fix: refs #7524 dynamic fetch #828

Merged
jorgep merged 3 commits from 7524-hotfix-dynamicFetch into master 2024-10-14 14:02:04 +00:00
Member
No description provided.
jorgep added 1 commit 2024-10-14 10:00:34 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
4b78c1acd0
fix: refs #7524 dynamic
jorgep requested review from jsegarra 2024-10-14 10:11:43 +00:00
jorgep reviewed 2024-10-14 10:12:13 +00:00
@ -15,25 +15,10 @@ const props = defineProps({
required: true,
Author
Member

No se usa

No se usa
jorgep added the
CR / Tests passed
label 2024-10-14 10:13:56 +00:00
jorgep changed title from fix: refs #7524 dynamic to fix: refs #7524 dynamic fetch 2024-10-14 10:17:21 +00:00
jsegarra requested changes 2024-10-14 10:29:25 +00:00
Dismissed
@ -63,2 +56,3 @@
:label="t('itemType.shared.worker')"
:options="workersOptions"
url="Workers"
sort-by="firstName ASC"
Member

aquí has puesto firstName ASC, pero en otro archivo de la PR has dejado firstName. Concretamente en ItemTypeBasicData

aquí has puesto firstName ASC, pero en otro archivo de la PR has dejado firstName. Concretamente en ItemTypeBasicData
jorgep marked this conversation as resolved
@ -72,2 +63,2 @@
<QItemSection v-if="workers">
<QSelect
<QItemSection>
<VnSelect
Member

Tenemos varias veces repetido este VnSelect, no?
Creo que al menos he visto 2 en esta PR, como de correcto seria hacer un componente aparte como si se hizo en salix, vn-worker-autocompletE?

Tenemos varias veces repetido este VnSelect, no? Creo que al menos he visto 2 en esta PR, como de correcto seria hacer un componente aparte como si se hizo en salix, vn-worker-autocompletE?
Author
Member

Lo veo bien, pero creo que mejor tarea, porque también está worker/search .

Lo veo bien, pero creo que mejor tarea, porque también está worker/search .
jorgep added 1 commit 2024-10-14 11:23:02 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
0f3168456a
chore: refs #7524 sort by firstName
jorgep requested review from jsegarra 2024-10-14 11:23:15 +00:00
jorgep added 1 commit 2024-10-14 13:52:17 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
2d34921cf5
fix: refs #7524 paginate
jsegarra reviewed 2024-10-14 13:57:14 +00:00
@ -228,2 +228,4 @@
const getVal = (val) => ($props.useLike ? { like: `%${val}%` } : val);
defineExpose({ opts: myOptions });
Member

Molaria que en la guia de estilos hubiese un apartado para defineExpose
https://vuejs.org/style-guide/rules-recommended
Lo dejamos asi

Molaria que en la guia de estilos hubiese un apartado para defineExpose https://vuejs.org/style-guide/rules-recommended Lo dejamos asi
jsegarra approved these changes 2024-10-14 13:57:16 +00:00
jorgep merged commit 12e1d9c389 into master 2024-10-14 14:02:04 +00:00
jorgep deleted branch 7524-hotfix-dynamicFetch 2024-10-14 14:02:04 +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#828
No description provided.