fix: refs #7356 create claim dialog #656

Merged
jorgep merged 6 commits from 7356-fixCreateClaimDialog into dev 2024-08-29 14:38:46 +00:00
Member
No description provided.
jorgep added 1 commit 2024-08-28 14:18:07 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
174c34b4e0
fix: refs #7356 create claim dialog
jorgep reviewed 2024-08-28 14:18:35 +00:00
@ -134,1 +134,3 @@
openConfirmationModal(t('Do you want to create a claim?'), onCreateClaimAccepted);
openConfirmationModal(
t('Do you want to create a claim?'),
null,
Author
Member

Tiene 3 parámetros obligatiorios.

Tiene 3 parámetros obligatiorios.
jorgep added the
CR / Tests passed
label 2024-08-28 14:28:39 +00:00
jorgep requested review from jgallego 2024-08-28 14:28:45 +00:00
jgallego approved these changes 2024-08-28 14:32:58 +00:00
Dismissed
jorgep added 1 commit 2024-08-28 15:01:57 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
23bf9657aa
fix: refs #7356 colors
jorgep dismissed jgallego’s review 2024-08-28 15:01:57 +00:00
Reason:

New commits pushed, approval review dismissed automatically according to repository settings

jorgep added 1 commit 2024-08-28 15:03:40 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
041b6561f8
fix: refs #7356 colors
jorgep requested review from jgallego 2024-08-28 15:05:23 +00:00
jorgep added 1 commit 2024-08-28 15:06:19 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
c4665f6cc1
chore: refs #7356 fix type
jgallego requested changes 2024-08-28 15:35:49 +00:00
Dismissed
@ -36,3 +36,3 @@
const title = props.title || t('Confirm');
const message = props.message || t('Are you sure you want to continue?');
const message =
Owner

la IA dice:

Esta versión mejorada hace lo siguiente:

Si props.message es undefined, usa el texto traducido por defecto.
En cualquier otro caso, usa el valor de props.message, incluso si es false, una cadena vacía, o cualquier otro valor.

Esta lógica es más clara y predecible:

Permite pasar explícitamente false o una cadena vacía si se desea que no haya mensaje.
Usa el mensaje por defecto solo cuando no se ha proporcionado ningún valor para props.message.

Esta versión evita la confusión potencial de la lógica original y hace que el comportamiento sea más consistente y fácil de entender.

la IA dice: Esta versión mejorada hace lo siguiente: Si props.message es undefined, usa el texto traducido por defecto. En cualquier otro caso, usa el valor de props.message, incluso si es false, una cadena vacía, o cualquier otro valor. Esta lógica es más clara y predecible: Permite pasar explícitamente false o una cadena vacía si se desea que no haya mensaje. Usa el mensaje por defecto solo cuando no se ha proporcionado ningún valor para props.message. Esta versión evita la confusión potencial de la lógica original y hace que el comportamiento sea más consistente y fácil de entender.
Author
Member

Pero, ayer me dijiste que lo quieres sin ningún subtítulo...(Do you want to continue?) Tal cual está en Salix.

Pero, ayer me dijiste que lo quieres sin ningún subtítulo...(Do you want to continue?) Tal cual está en Salix.
jorgep added 1 commit 2024-08-29 07:24:40 +00:00
jorgep requested review from jgallego 2024-08-29 10:22:36 +00:00
jgallego approved these changes 2024-08-29 13:05:02 +00:00
jorgep added 1 commit 2024-08-29 14:36:29 +00:00
jorgep merged commit a845190768 into dev 2024-08-29 14:38:46 +00:00
jorgep deleted branch 7356-fixCreateClaimDialog 2024-08-29 14:38:46 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 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#656
No description provided.