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

#8409: Added VnSelectSupplier #1274

Merged
jon merged 10 commits from 8409-VnSelectSupplier into dev 2025-01-30 14:20:07 +00:00
Conversation 1 Commits 10 Files Changed 15 -8
1 changed files with 0 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit ed211ba14b - Show all commits

8
src/components/common/VnSelectSupplier.vue
View File

@ -4,14 +4,6 @@ import VnSelect from 'components/common/VnSelect.vue';
const model = defineModel({ type: [String, Number, Object] });
const url = 'Suppliers';
jon marked this conversation as resolved
jsegarra commented 2025-01-27 22:04:35 +00:00
Review
Copy Link

nos hace falta defineProps? Porque no defineModel?
Al actualziar el valor ya emite no?
Me vaso para este comentario en VnInputNumber o VnRadio

nos hace falta defineProps? Porque no defineModel? Al actualziar el valor ya emite no? Me vaso para este comentario en VnInputNumber o VnRadio
jsegarra commented 2025-01-28 11:59:46 +00:00
Review
Copy Link

Okey @jon, queda resolver esto

Okey @jon, queda resolver esto
const value = computed({
get() {
return model;
},
set(val) {
emit('update:modelValue', val);
},
});
</script>
<template>
jsegarra commented 2025-01-29 22:00:55 +00:00
Outdated
Review
Copy Link

pero...le estamos dando uso a la variable value?

pero...le estamos dando uso a la variable value?