#6896 fix Order module problems #817

Merged
jon merged 18 commits from Fix_OrderModuleProblems into dev 2024-10-29 11:20:14 +00:00
Member

Fixed problems in catalog, lines, descriptor and summary(dialog)

Commit del back donde se eliminan los archivos: 3d69d1ad62
Utilizarlo para generar tests e2e

Fixed problems in catalog, lines, descriptor and summary(dialog) Commit del back donde se eliminan los archivos: https://gitea.verdnatura.es/verdnatura/salix/commit/3d69d1ad626be4ae654b621669ea23a760ab4a2a Utilizarlo para generar tests e2e
jon added 1 commit 2024-10-10 05:21:05 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
419d3d2d45
fix: refs #6896 fixed module problems
jon requested review from jsegarra 2024-10-10 05:35:30 +00:00
jon added 1 commit 2024-10-10 06:43:40 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
d721083795
fix: filter panel
jsegarra added 2 commits 2024-10-10 07:30:02 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
3d5d267345
revert: commit
jsegarra requested changes 2024-10-10 09:07:25 +00:00
Dismissed
@ -9,6 +9,7 @@ import { useStateStore } from 'src/stores/useStateStore';
const quasar = useQuasar();
const { t } = useI18n();
const state = useStateStore();
const emit = defineEmits(['on-search']);
Member

No sigue el standard

No sigue el standard
jon marked this conversation as resolved
@ -49,2 +48,4 @@
const orderWaySelected = ref('ASC');
const routeQuery = JSON.parse(route?.query.params ?? '{}');
const params = ref({});
Member

puede confundirse con la variable de la 189

puede confundirse con la variable de la 189
jon marked this conversation as resolved
@ -51,0 +51,4 @@
const params = ref({});
onMounted(() => {
const filter = routeQuery.filter;
Member

si solo lo usas una vez porque no eliminas esta variable y lo pones en la siguiente linea? si tampoco haces ninguna validación

si solo lo usas una vez porque no eliminas esta variable y lo pones en la siguiente linea? si tampoco haces ninguna validación
jon marked this conversation as resolved
@ -193,2 +176,4 @@
{{ t(selectedType?.name || '') }}
</strong>
<div v-else-if="tag.label === 'tagGroups'" class="q-gutter-x-xs">
<strong v-if="JSON.parse(tag.value).tagSelection.name"
Member

podrias hacer una funcion o computed para JSON.parse

podrias hacer una funcion o computed para JSON.parse
jon added 2 commits 2024-10-18 12:12:32 +00:00
jon added 2 commits 2024-10-22 10:06:09 +00:00
jon requested review from jsegarra 2024-10-22 10:06:59 +00:00
jsegarra requested changes 2024-10-22 22:55:52 +00:00
Dismissed
@ -86,11 +86,8 @@ function extractValueTags(items) {
<div class="full-width">
Member

Añadimos clearable al input de cuando le das al mas

Añadimos clearable al input de cuando le das al mas
@ -1,14 +1,13 @@
<script setup>
Member

Nos juntamos porque creo que me siguen sin funcionar como es debido cuando refresco la ventana.
Se mantiene categoria, pero si seleccionas tipo y F5, se pierde todo.
El tema de las etiquetas o tagGroups, no aparece al refrescar

Si lo ves muy complejo se lo pasamos a William

Nos juntamos porque creo que me siguen sin funcionar como es debido cuando refresco la ventana. Se mantiene categoria, pero si seleccionas tipo y F5, se pierde todo. El tema de las etiquetas o tagGroups, no aparece al refrescar Si lo ves muy complejo se lo pasamos a William
jon added 1 commit 2024-10-23 11:02:41 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
37b3affdbf
refactor: revert catalog changes
jon requested review from jsegarra 2024-10-23 11:05:06 +00:00
jon added 1 commit 2024-10-23 11:05:16 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
76ed5150a5
Merge branch 'dev' into Fix_OrderModuleProblems
jsegarra requested changes 2024-10-28 09:38:03 +00:00
Dismissed
@ -50,2 +56,4 @@
},
]);
async function confirmOrder() {
Member

Tenemos duplicidad al usar este axios.post, lo movemos un archivo global.
si tienes dudas, puedes revisar setRectificative

Tenemos duplicidad al usar este axios.post, lo movemos un archivo global. si tienes dudas, puedes revisar setRectificative
jon added 2 commits 2024-10-28 11:21:35 +00:00
jon added 1 commit 2024-10-28 11:22:26 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
639e08406e
Merge branch 'dev' into Fix_OrderModuleProblems
jon requested review from jsegarra 2024-10-28 11:29:46 +00:00
jsegarra requested changes 2024-10-28 11:47:33 +00:00
Dismissed
@ -0,0 +2,4 @@
import { useQuasar } from 'quasar';
import { useI18n } from 'vue-i18n';
export function confirmOrder() {
Member

Esta logica la tiene mas partes de la aplicacion?

Si es que no, moverlo al modulo que lo esté usando en el momento
Si es que si, revis si sigue la nomenclatura correcta

Esta logica la tiene mas partes de la aplicacion? Si es que **no**, moverlo al modulo que lo esté usando en el momento Si es que **si**, revis si sigue la nomenclatura correcta
jon added 2 commits 2024-10-28 11:59:38 +00:00
jon requested review from jsegarra 2024-10-28 12:07:13 +00:00
jsegarra approved these changes 2024-10-29 01:43:31 +00:00
Dismissed
jsegarra reviewed 2024-10-29 01:45:05 +00:00
@ -0,0 +2,4 @@
import { useQuasar } from 'quasar';
import { useI18n } from 'vue-i18n';
export function confirmOrder() {
Member

No me acaba de convencer este archivo porque solo tenemos una función dentro, y a futuro no creo que escale
Yo lo he enfocado de esta manera
987c28e745/src/pages/Customer/composables/getClientRisk.js

No me acaba de convencer este archivo porque solo tenemos una función dentro, y a futuro no creo que escale Yo lo he enfocado de esta manera https://gitea.verdnatura.es/verdnatura/salix-front/src/commit/987c28e745fde447edb45b0223bc9e6a6c9f47e4/src/pages/Customer/composables/getClientRisk.js
jon added 1 commit 2024-10-29 08:23:57 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
5663064dd8
refactor: modified composable
jon dismissed jsegarra’s review 2024-10-29 08:23:57 +00:00
Reason:

New commits pushed, approval review dismissed automatically according to repository settings

jon added 1 commit 2024-10-29 08:26:18 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
1ff77abaf5
Merge branch 'dev' into Fix_OrderModuleProblems
jon requested review from jsegarra 2024-10-29 08:53:53 +00:00
jsegarra approved these changes 2024-10-29 09:23:32 +00:00
jon added 1 commit 2024-10-29 11:19:24 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
9104e13a6d
Merge branch 'dev' into Fix_OrderModuleProblems
jon merged commit 89dfc0aff5 into dev 2024-10-29 11:20:14 +00:00
jon deleted branch Fix_OrderModuleProblems 2024-10-29 11:20:14 +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#817
No description provided.