#8018 - Show fields instead null/undefined #737

Merged
jsegarra merged 9 commits from 8018_location_label into dev 2024-09-30 10:55:05 +00:00
Member

image

![image](/attachments/fe0685bf-216d-4cd0-82e7-f8fc79f270d2)
133 KiB
jsegarra added 1 commit 2024-09-20 10:01:48 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
271e33a999
feat: formatLocation when field is null
jsegarra changed title from WIP: Show fields to Show fields 2024-09-20 11:08:21 +00:00
jsegarra changed title from Show fields to #8018 - Show fields instead null/undefined 2024-09-20 11:08:45 +00:00
jsegarra requested review from alexm 2024-09-20 11:08:47 +00:00
Author
Member

image

![image](/attachments/560ae300-18ed-44d0-97f0-6edd6aa1d92b)
jsegarra added 2 commits 2024-09-23 09:48:08 +00:00
gitea/salix-front/pipeline/pr-dev Build queued... Details
013a76cbce
test: VnLocation
jsegarra added 1 commit 2024-09-23 09:48:21 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
7713e33c67
Merge branch 'dev' into 8018_location_label
jsegarra added 1 commit 2024-09-26 12:48:51 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
428c6966f2
Merge branch 'dev' into 8018_location_label
jsegarra requested review from jgallego 2024-09-27 11:49:32 +00:00
jgallego requested changes 2024-09-27 11:52:11 +00:00
Dismissed
@ -15,0 +35,4 @@
obj.city
? `${obj.city}${obj.province?.name ? `(${obj.province.name})` : ''}`
: null,
(obj) => obj.country?.name,
Owner

mostrar el pais si esta informado

mostrar el pais si esta informado
Author
Member

@jgallego El pais se está informado correctamente si se ha indicado. Adjunto una captura de como está definido el proveedor con ciudad y countryFk
Adjunto también lo que ocurre cuando no hay pais informado

### Conclusión: en la fixture no estaba puesto el pais

@jgallego El pais se está informado correctamente ***si** se ha indicado*. Adjunto una captura de como está definido el proveedor con ciudad y countryFk Adjunto también lo que ocurre cuando no hay pais informado **### Conclusión: en la fixture no estaba puesto el pais**
Author
Member

Con countryFK: image
Sin countryFK: image

Con countryFK: ![image](/attachments/65d42596-9ce6-4ccf-9e20-2ffbb126293c) Sin countryFK: ![image](/attachments/a2f02255-737f-4bf7-b399-c5eefbfcece9)
159 KiB
121 KiB
jsegarra requested review from jgallego 2024-09-27 18:01:13 +00:00
jsegarra added 1 commit 2024-09-27 18:01:19 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
cffd97a521
Merge branch 'dev' into 8018_location_label
jgallego approved these changes 2024-09-28 08:19:18 +00:00
jsegarra closed this pull request 2024-09-28 09:46:58 +00:00
jsegarra reopened this pull request 2024-09-28 09:47:04 +00:00
jsegarra added 2 commits 2024-09-30 07:43:11 +00:00
alexm approved these changes 2024-09-30 08:23:19 +00:00
jsegarra added 1 commit 2024-09-30 10:53:35 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
cf509d608a
Merge branch 'dev' into 8018_location_label
jsegarra merged commit d4dc12ebe3 into dev 2024-09-30 10:55:05 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: verdnatura/salix-front#737
No description provided.