verdnatura/salix-front
verdnatura
/
salix-front
2
3
Fork 1
Code Issues Pull Requests 88 Releases Wiki Activity

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
Conversation 0 Commits 6 Files Changed 2 +12 -5
2 changed files with 12 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/components/ui/VnConfirm.vue
View File

@ -15,7 +15,7 @@ const props = defineProps({
default: null, default: null,
}, },
message: { message: {
type: String, type: [String, Boolean],
default: null, default: null,
}, },
data: { data: {
@ -35,7 +35,10 @@ defineEmits(['confirm', ...useDialogPluginComponent.emits]);
const { dialogRef, onDialogOK } = useDialogPluginComponent(); const { dialogRef, onDialogOK } = useDialogPluginComponent();
const title = props.title || t('Confirm'); const title = props.title || t('Confirm');
const message = props.message || t('Are you sure you want to continue?'); const message =
jgallego commented 2024-08-28 15:35:06 +00:00
Review
Copy Link

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.
jorgep commented 2024-08-29 07:26:32 +00:00
Review
Copy Link

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.
props.message ||
(props.message !== false ? t('Are you sure you want to continue?') : false);
const isLoading = ref(false); const isLoading = ref(false);
async function confirm() { async function confirm() {
@ -61,12 +64,12 @@ async function confirm() {
size="xl" size="xl"
v-if="icon" v-if="icon"
/> />
<span class="text-h6 text-grey">{{ title }}</span> <span class="text-h6">{{ title }}</span>
<QSpace /> <QSpace />
<QBtn icon="close" :disable="isLoading" flat round dense v-close-popup /> <QBtn icon="close" :disable="isLoading" flat round dense v-close-popup />
</QCardSection> </QCardSection>
<QCardSection class="row items-center"> <QCardSection class="row items-center">
<span v-html="message"></span> <span v-if="message !== false" v-html="message" />
<slot name="customHTML"></slot> <slot name="customHTML"></slot>
</QCardSection> </QCardSection>
<QCardActions align="right"> <QCardActions align="right">

6
src/pages/Ticket/Card/TicketSaleMoreActions.vue
View File

@ -131,7 +131,11 @@ const createClaim = () => {
onCreateClaimAccepted onCreateClaimAccepted
); );
else else
openConfirmationModal(t('Do you want to create a claim?'), onCreateClaimAccepted); openConfirmationModal(
t('Do you want to create a claim?'),
false,
jorgep commented 2024-08-28 14:18:35 +00:00
Outdated
Review
Copy Link

Tiene 3 parámetros obligatiorios.

Tiene 3 parámetros obligatiorios.
onCreateClaimAccepted
);
}; };
const onCreateClaimAccepted = async () => { const onCreateClaimAccepted = async () => {