#7717 fix OrderCatalog #558

Merged
jon merged 39 commits from 7717_fixOrderCatalog into dev 2024-08-19 09:22:35 +00:00
Member
No description provided.
jon added 1 commit 2024-07-19 12:54:17 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
7f376c8ea4
fix: refs #7717 fix catalog filter, searchbar redirect and search
jon added 1 commit 2024-07-19 12:55:26 +00:00
jon added 1 commit 2024-07-19 13:00:13 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
02e1de083a
fix: duplicate key
jon added 1 commit 2024-07-22 10:53:08 +00:00
jon added 1 commit 2024-07-22 10:53:58 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
848b267112
Merge branch 'dev' into 7717_fixOrderCatalog
jsegarra added 1 commit 2024-07-22 13:13:07 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
368b8404fa
updates: OrderCatlago
jon added 1 commit 2024-07-23 09:46:29 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
ce6a94bab8
fix: refs #7717 fix catalog searchbar and worker tests(refs #7323)
jon changed title from WIP: #7717 fix OrderCatalog to #7717 fix OrderCatalog 2024-07-23 10:03:14 +00:00
jon requested review from jsegarra 2024-07-23 10:03:21 +00:00
jon requested review from alexm 2024-07-23 10:21:23 +00:00
jsegarra added 1 commit 2024-07-23 13:09:48 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
1ac0a85969
updates
jsegarra added 1 commit 2024-07-23 18:18:58 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
e013e5571d
perf: use ref at component start
jon added 1 commit 2024-07-24 08:33:31 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
cc518d5a80
perf: refs #7717 right menu filter
jon added 1 commit 2024-07-24 08:35:26 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
3a9116987c
Merge branch 'dev' into 7717_fixOrderCatalog
jon added 1 commit 2024-07-25 08:07:29 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
2de11d3ecf
Merge branch 'dev' into 7717_fixOrderCatalog
jon added 2 commits 2024-07-26 12:09:16 +00:00
jon added 1 commit 2024-07-26 12:11:20 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
356e7768b7
Merge branch 'dev' into 7717_fixOrderCatalog
jon added 2 commits 2024-07-26 12:56:33 +00:00
jon added 1 commit 2024-07-30 06:03:53 +00:00
jon added 1 commit 2024-07-30 11:31:20 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
7e3bfce732
fix: refs #7717 fix basic data form & minor errors
jon added 1 commit 2024-07-30 11:32:07 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
610e7c150b
Merge branch 'dev' into 7717_fixOrderCatalog
alexm requested changes 2024-07-30 12:14:46 +00:00
Dismissed
@ -34,1 +34,3 @@
const searchRightDataKey = computed(() => {
if (!props.searchDataKey) return route.name;
else return props.searchDataKey;
Member

Else no hace falta

Else no hace falta
jon marked this conversation as resolved
@ -6,0 +12,4 @@
const route = useRoute();
const routeName = computed(() => route.name);
const customRouteRedirectName = computed(() => {
Member

Esto para que hace falta?

Esto para que hace falta?
Author
Member

Para eliminar el duplicado de la searchbar, ya que la searchbar de catalog es distinta al del resto del módulo

Para eliminar el duplicado de la searchbar, ya que la searchbar de catalog es distinta al del resto del módulo
@ -107,4 +106,2 @@
}
};
async function onDataSaved({ id }) {
Member

Ya no se quiere?

Ya no se quiere?
Author
Member

Lo puse yo porque durante la migración no se tuvo en cuenta la validación del back. Tras acabarla he visto que no hacía falta y que poniendo :url-update funcionaba

Lo puse yo porque durante la migración no se tuvo en cuenta la validación del back. Tras acabarla he visto que no hacía falta y que poniendo :url-update funcionaba
alexm marked this conversation as resolved
@ -0,0 +197,4 @@
});
}
watch(
Member

Crec que vaig ficar una cosa per a que detectara els canvis del filter, comprova si encara cal este codi

Crec que vaig ficar una cosa per a que detectara els canvis del filter, comprova si encara cal este codi
Author
Member

He probado a eliminarlo y sin el watch solo cambia lo que muestra el descriptor

He probado a eliminarlo y sin el watch solo cambia lo que muestra el descriptor
alexm marked this conversation as resolved
@ -0,0 +38,4 @@
});
};
watch(
Member

Crec que vaig ficar una cosa per a que detectara els canvis del filter, comprova si encara cal este codi

Crec que vaig ficar una cosa per a que detectara els canvis del filter, comprova si encara cal este codi
jon added 3 commits 2024-07-31 05:36:11 +00:00
jon requested review from alexm 2024-07-31 05:37:09 +00:00
jon added 1 commit 2024-08-02 04:50:41 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
8a4f723732
Merge branch 'dev' into 7717_fixOrderCatalog
alexm requested changes 2024-08-05 05:31:06 +00:00
Dismissed
alexm left a comment
Member

En salix en la cesta 4 en /catalogo te redirigis a /lineas
En /lineas donem la opcio de borrar la linea, pero en salix no. Pariex que es per el estat, revisar el codi
Si entres a /Volumen despres de haver entrat en lineas, la interfaz se ralla
En /Volumen pq fica summary? En salix no ho ficava
En /BasicData per example en la 1, en el camp address, ha de ficar "the bat cave" i no "address 21"
En /Catalogo en el 7. No deuria de tindre auto-load
En /Catalogo en el 7. Al afegir un item deuria refrescar el descriptor (arrayData)

En salix en la cesta 4 en /catalogo te redirigis a /lineas En /lineas donem la opcio de borrar la linea, pero en salix no. Pariex que es per el estat, revisar el codi Si entres a /Volumen despres de haver entrat en lineas, la interfaz se ralla En /Volumen pq fica summary? En salix no ho ficava En /BasicData per example en la 1, en el camp address, ha de ficar "the bat cave" i no "address 21" En /Catalogo en el 7. No deuria de tindre auto-load En /Catalogo en el 7. Al afegir un item deuria refrescar el descriptor (arrayData)
Member

El popup deuria opcupar mes ample
image

El popup deuria opcupar mes ample ![image](/attachments/6f5b6004-f1de-43ca-903d-af765714650f)
176 KiB
Member

En salix i lilium no mostra lo mateix al buscar per "2" en la barra de busqueda en catalogo, apart fa coses rares en els panells laterals:
Lilium
image
Salix
image

En salix i lilium no mostra lo mateix al buscar per "2" en la barra de busqueda en catalogo, apart fa coses rares en els panells laterals: Lilium ![image](/attachments/66a2d459-282a-4d4a-85c1-231055017d07) Salix ![image](/attachments/a219b518-9e08-4368-8417-f87eef88284b)
162 KiB
111 KiB
jon added 3 commits 2024-08-09 05:46:41 +00:00
jon requested review from alexm 2024-08-09 06:04:31 +00:00
alexm added 1 commit 2024-08-09 06:52:15 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
d3f50ec9b2
Merge branch 'dev' into 7717_fixOrderCatalog
alexm requested changes 2024-08-09 06:53:50 +00:00
Dismissed
alexm left a comment
Member

El filtro de la derecha del list y dentro deberia ser el mismo
Quitar auto-load
Al crear una cesta, en Salix los datos de las agencias los saca de landsThatDay
Al crear te debe llevar al catalogo directamente
En basicData falta el camp de notas
En catalogo el filtro lateral faltan traducciones en Ordenar por y orden
El list no ordena igual que en salix
Volume pasar VnTable, con card ocupa demasiado
EN lineas si borres ha de recardar el panel lateral dret
En lineas al confirmar te debe llevar al list filtrando por ese cliente

El filtro de la derecha del list y dentro deberia ser el mismo Quitar auto-load Al crear una cesta, en Salix los datos de las agencias los saca de landsThatDay Al crear te debe llevar al catalogo directamente En basicData falta el camp de notas En catalogo el filtro lateral faltan traducciones en Ordenar por y orden El list no ordena igual que en salix Volume pasar VnTable, con card ocupa demasiado EN lineas si borres ha de recardar el panel lateral dret En lineas al confirmar te debe llevar al list filtrando por ese cliente
alexm added 2 commits 2024-08-09 10:39:07 +00:00
alexm added 1 commit 2024-08-09 11:07:43 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
ee71ae9d24
feat(orderList): use orderFilter and fixed this
alexm added 1 commit 2024-08-09 12:19:15 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
dc7c686508
feat(orderList): correct create order
alexm added 1 commit 2024-08-09 12:55:57 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
37244cdbc7
feat(orderBasicData): add notes
alexm added 1 commit 2024-08-12 07:17:59 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
e11e51f432
fix: order description to vnTable
alexm added 1 commit 2024-08-12 08:23:05 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
190579e7fd
fix: orderCatalogFilter order
alexm added 1 commit 2024-08-12 09:47:18 +00:00
alexm added 1 commit 2024-08-12 09:51:29 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
3222a7a42c
fix(orderLines): reload when delete and redirect when confirm
alexm approved these changes 2024-08-12 10:51:24 +00:00
Dismissed
jsegarra requested changes 2024-08-19 08:31:08 +00:00
Dismissed
@ -0,0 +64,4 @@
fetchAgencyList(order?.landed, order?.addressFk);
};
const orderMapper = (order) => {
Member

esta función está sin uso

esta función está sin uso
jsegarra requested changes 2024-08-19 08:32:12 +00:00
Dismissed
@ -9,6 +9,7 @@ import VnSelect from 'components/common/VnSelect.vue';
import VnFilterPanelChip from 'components/ui/VnFilterPanelChip.vue';
import { useValidator } from 'src/composables/useValidator';
Member

useValidator no se usa en la template ni en script-setup

useValidator no se usa en la template ni en script-setup
jon added 1 commit 2024-08-19 08:49:12 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
f3d7590edf
refactor: refs #7717 delete useless function and import
jon dismissed alexm’s review 2024-08-19 08:49:12 +00:00
Reason:

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

jon requested review from jsegarra 2024-08-19 09:20:25 +00:00
jon added 1 commit 2024-08-19 09:20:33 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
ce8d12fe20
Merge branch 'dev' into 7717_fixOrderCatalog
jsegarra approved these changes 2024-08-19 09:21:07 +00:00
jon merged commit 5230b8f722 into dev 2024-08-19 09:22:35 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
3 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#558
No description provided.