verdnatura/salix-front
verdnatura
/
salix-front
2
3
Fork 1
Code Issues Pull Requests 70 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 +3 -9
1 changed files with 3 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 613ebc0eb4 - Show all commits

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

@ -2,17 +2,11 @@
import { computed } from 'vue'; import { computed } from 'vue';
import VnSelect from 'components/common/VnSelect.vue'; import VnSelect from 'components/common/VnSelect.vue';
const emit = defineEmits(['update:modelValue']); const model = defineModel({ type: [String, Number, Object] });
const $props = defineProps({
modelValue: {
type: [String, Number, Object],
default: null,
},
});
const url = 'Suppliers'; 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({ const value = computed({
get() { get() {
return $props.modelValue; return model;
}, },
set(val) { set(val) {
emit('update:modelValue', val); emit('update:modelValue', val);
@ -24,7 +18,7 @@ const value = computed({
<VnSelect <VnSelect
:label="$t('globals.supplier')" :label="$t('globals.supplier')"
v-bind="$attrs" v-bind="$attrs"
v-model="value" v-model="model"
:url="url" :url="url"
option-value="id" option-value="id"
option-label="nickname" option-label="nickname"