#7136 - Enable paginate event in VnSelectFilter #255

Closed
jsegarra wants to merge 86 commits from 7136_vnselectFilter_paginate into dev
3 changed files with 25 additions and 17 deletions
Showing only changes of commit a23eb0a5ad - Show all commits

View File

@ -36,10 +36,10 @@ const $props = defineProps({
const emit = defineEmits(['onFetch']);
defineExpose({ fetch });
const fetchData = useFetchData({ ...$props });
onMounted(async () => {
Outdated
Review

No estoy segur de si deberiamos poner la funcionalidad de usar arrayData//paginar en FechData si luego quien lo usa es VnSelectFilter, en vez de ponerla directamente en VnSelectFilter. @juan

No estoy segur de si deberiamos poner la funcionalidad de usar arrayData//paginar en FechData si luego quien lo usa es VnSelectFilter, en vez de ponerla directamente en VnSelectFilter. @juan
if ($props.autoLoad) {
await useFetchData(...$props).fetch();
await fetchData.fetch();
}
});

View File

@ -2,7 +2,6 @@
import { ref, toRefs, computed, watch, h } from 'vue';
import { onMounted } from 'vue';
import { useI18n } from 'vue-i18n';
import FetchData from 'src/components/FetchData.vue';
const emit = defineEmits(['update:modelValue', 'update:options']);
import { useArrayData } from 'src/composables/useArrayData';
import { useFetchData } from 'src/composables/useFetchData';
jsegarra marked this conversation as resolved Outdated
Outdated
Review

Quitar

Quitar
@ -78,18 +77,6 @@ const dataRef = ref(
fields: $props.fields,
})
);
// const fetchDataComponent = ref(null);
// fetchDataComponent.value = h(FetchData, {
// url: $props.url,
// where: $props.where,
// limit: $props.limit,
// sortBy: $props.sortBy,
// fields: $props.fields,
// onFetch: (data) => {
// setOptions(data); // Tu función para manejar los datos
// },
// });
// const dataRef = ref(null);
const selectValue = computed({
get() {
return $props.modelValue;
@ -108,10 +95,10 @@ const useURL = computed(() => $props.url?.length > 0);
const arrayData = useURL.value
? useArrayData($props.url, {
url: $props.url,
fields: $props.fields,
where: $props.where,
jsegarra marked this conversation as resolved Outdated
Outdated
Review

Si lo estas definiendo no puede ser null no?

Si lo estas definiendo no puede ser null no?
sortBy: $props.sortBy,
limit: $props.limit,
sortBy: $props.sortBy,
fields: $props.fields,
})
: ref(null);
onMounted(async () => {

View File

@ -0,0 +1,21 @@
import axios from 'axios';
export function useAxios() {
const fetch = async (url, filter, params) => {
axios.get(url, { params: { ...params, ...filter } });
};
const push = async (url, method = 'PUT', filter, params) => {
axios({ url, method, params: { ...params, ...filter } });
};
const pop = async (url, filter, params) => {
Outdated
Review

get no?

get no?

Lo he eliminado de la PR, y un ejemplo de su implementacion tambien.

Asi se queda lo mas limpio posible.
cuando surja la tarea si hace falta hacemos cherry-pick

Lo he eliminado de la PR, y un ejemplo de su implementacion tambien. Asi se queda lo mas limpio posible. cuando surja la tarea si hace falta hacemos cherry-pick
axios.delete(url, { params: { ...params, ...filter } });
};
return {
fetch,
push,
pop,
};
}