verdnatura/salix-front
verdnatura
/
salix-front
2
3
Fork 1
Code Issues Pull Requests 77 Releases Wiki Activity

refactor: #7516 unified navigate and redirectTo functions #1324

Closed
provira wants to merge 8 commits from 7516-refactorNavigateAndRedirect into dev
pull from: 7516-refactorNavigateAndRedirect
merge into: verdnatura:dev
Conversation 1 Commits 8 Files Changed 11 +46 -51
provira commented 2025-02-03 09:27:38 +00:00
Member
Copy Link
No description provided.
provira added 1 commit 2025-02-03 09:27:39 +00:00
provira added 1 commit 2025-02-03 09:27:46 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
b463baf8d5 Merge branch 'dev' into 7516-refactorNavigateAndRedirect
provira requested review from jsegarra 2025-02-03 09:29:59 +00:00
jsegarra requested changes 2025-02-04 22:35:04 +00:00
src/composables/useNavigate.js
@ -0,0 +2,4 @@
export function useNavigate() {
const router = useRouter();
const route = useRoute();
jsegarra commented 2025-02-04 22:26:58 +00:00
Member
Copy Link

se usa route?
Si haces quasar build saldrá error, no?

se usa route? Si haces quasar build saldrá error, no?
provira commented 2025-03-14 13:01:41 +00:00
Author
Member
Copy Link

Si, se usaba en una modificación anterior pero no se quitó al cambiarse

Si, se usaba en una modificación anterior pero no se quitó al cambiarse
👍 1
src/pages/Route/Agency/AgencyList.vue Outdated
@ -62,3 +61,3 @@
title: t('Client ticket list'),
icon: 'preview',
action: (row) => navigate(row.id),
action: (row) => redirectTo({ path: '/route/agency/'+row.id}),
jsegarra commented 2025-02-04 22:33:28 +00:00
Member
Copy Link

Porque no usar template String si ya lo tenias antes?
Quiero decir, porque tiene otro formato?
Otra manera de hacerlo es con name y params, que he visto que has usado mas abajo
Unificar criterio?? Creo que nos centramos en dejarlo como estaba

Porque no usar template String si ya lo tenias antes? Quiero decir, porque tiene otro formato? Otra manera de hacerlo es con name y params, que he visto que has usado mas abajo Unificar criterio?? Creo que nos centramos en dejarlo como estaba
provira commented 2025-03-14 13:05:21 +00:00
Author
Member
Copy Link

Estoy contemplando usar name y params en todos, en vez de path

Estoy contemplando usar name y params en todos, en vez de path
jsegarra commented 2025-03-16 09:05:50 +00:00
Member
Copy Link

me parece bien
No le dediquemos mucho mas tiempo.
Si ves que es agil, adelante

me parece bien No le dediquemos mucho mas tiempo. Si ves que es agil, adelante
src/pages/Route/RouteExtendedList.vue Outdated
@ -213,3 +214,3 @@
title: t('route.Route summary'),
icon: 'arrow_forward',
action: (row) => navigate(row?.id),
action: (row) => redirectTo({path: '/route/'+row.id}),
jsegarra commented 2025-02-04 22:33:21 +00:00
Member
Copy Link

Porque no usar template String si ya lo tenias antes?
Quiero decir, porque tiene otro formato?
Otra manera de hacerlo es con name y params, que he visto que has usado mas abajo
Unificar criterio?? Creo que nos centramos en dejarlo como estaba

Porque no usar template String si ya lo tenias antes? Quiero decir, porque tiene otro formato? Otra manera de hacerlo es con name y params, que he visto que has usado mas abajo Unificar criterio?? Creo que nos centramos en dejarlo como estaba
src/pages/Shelving/ShelvingList.vue Outdated
@ -59,3 +55,3 @@
:id="row.id"
:title="row.code"
@click="navigate(row.id)"
@click="redirectTo({path: '/shelving/'+row.id})"
jsegarra commented 2025-02-04 22:33:03 +00:00
Member
Copy Link

Porque no usar template String si ya lo tenias antes?
Quiero decir, porque tiene otro formato?
Otra manera de hacerlo es con name y params, que he visto que has usado mas abajo
Unificar criterio?? Creo que nos centramos en dejarlo como estaba

Porque no usar template String si ya lo tenias antes? Quiero decir, porque tiene otro formato? Otra manera de hacerlo es con name y params, que he visto que has usado mas abajo Unificar criterio?? Creo que nos centramos en dejarlo como estaba
src/pages/Travel/TravelList.vue Outdated
@ -36,3 +32,1 @@
const redirectCreateEntryView = (travelData) => {
router.push({ name: 'EntryCreate', query: { travelFk: travelData.id } });
redirectTo({ name: 'TravelCreate', query: { travelData: JSON.stringify(travelData) } });
jsegarra commented 2025-02-04 22:30:19 +00:00
Member
Copy Link

Tiene diferente formato al archivo anterior.
Unificar criterio

Tiene diferente formato al archivo anterior. Unificar criterio
src/pages/Wagon/WagonList.vue Outdated
@ -59,3 +60,3 @@
icon: 'arrow_forward',
isPrimary: true,
action: (row) => navigate(row.id),
action: (row) => redirectTo({ path: `/wagon/`+row.id+`/edit` }),
jsegarra commented 2025-02-04 22:34:22 +00:00
Member
Copy Link

Porque no usar template String si ya lo tenias antes?
Quiero decir, porque tiene otro formato?
Otra manera de hacerlo es con name y params, que he visto que has usado mas abajo
Unificar criterio?? Creo que nos centramos en dejarlo como estaba

Porque no usar template String si ya lo tenias antes? Quiero decir, porque tiene otro formato? Otra manera de hacerlo es con name y params, que he visto que has usado mas abajo Unificar criterio?? Creo que nos centramos en dejarlo como estaba
src/pages/Zone/Delivery/ZoneDeliveryList.vue
@ -55,3 +53,3 @@
:title="(row.name || '').toString()"
:id="row.id"
@click="navigate(row.id)"
@click="redirectTo({path: '/zone/type/'+row.id+'/edit'})"
jsegarra commented 2025-02-04 22:34:51 +00:00
Member
Copy Link

Porque no usar template String si ya lo tenias antes?
Quiero decir, porque tiene otro formato?
Otra manera de hacerlo es con name y params, que he visto que has usado mas abajo
Unificar criterio?? Creo que nos centramos en dejarlo como estaba

Porque no usar template String si ya lo tenias antes? Quiero decir, porque tiene otro formato? Otra manera de hacerlo es con name y params, que he visto que has usado mas abajo Unificar criterio?? Creo que nos centramos en dejarlo como estaba
src/pages/Zone/Upcoming/ZoneUpcomingList.vue
@ -60,3 +58,3 @@
<QBtn
:label="t('components.smartCard.openCard')"
@click.stop="navigate(row.id)"
@click.stop="redirectTo({path: '/zone/type/'+row.id+'/edit'})""
jsegarra commented 2025-02-04 22:35:02 +00:00
Member
Copy Link

Porque no usar template String si ya lo tenias antes?
Quiero decir, porque tiene otro formato?
Otra manera de hacerlo es con name y params, que he visto que has usado mas abajo
Unificar criterio?? Creo que nos centramos en dejarlo como estaba

Porque no usar template String si ya lo tenias antes? Quiero decir, porque tiene otro formato? Otra manera de hacerlo es con name y params, que he visto que has usado mas abajo Unificar criterio?? Creo que nos centramos en dejarlo como estaba
provira added 1 commit 2025-02-25 13:49:35 +00:00
provira added 1 commit 2025-03-05 12:11:57 +00:00
provira added 1 commit 2025-03-05 13:07:30 +00:00
gitea/salix-front/pipeline/pr-dev This commit is unstable Details
2102fbcc84 Merge branch 'dev' into 7516-refactorNavigateAndRedirect
provira added 2 commits 2025-03-26 10:21:32 +00:00
jsegarra added 1 commit 2025-04-04 07:19:03 +00:00
gitea/salix-front/pipeline/pr-dev Build started... Details
2606929503 Merge branch 'dev' into 7516-refactorNavigateAndRedirect
jsegarra commented 2025-04-14 15:56:51 +00:00
Member
Copy Link

La idea del conmposable o de la tarea era elimina la función(que eso ya está).
La idea era que si la función es "redirectToCreateView", la función tenga la lógica necesaria para llevar al path de crear. Cuando se hizo esta tarea todavía no existía createForm en VnTable, por lo que la lógica de la función debería morir ahí, de esta manera no haríamos siempre, createForm

Por otra parte navigate, por norma general te lleva a la card de ese módulo. La idea era usar useRoute.meta.moduleName para llevarte a la card de ese módulo.

Mañana lo vemos y revisamos el alcance de la tarea porque a lo mejor hacer router.push es suficiente aunque para crear no es tan sencillo.

La idea del conmposable o de la tarea era elimina la función(que eso ya está). La idea era que si la función es "redirectToCreateView", la función tenga la lógica necesaria para llevar al path de crear. Cuando se hizo esta tarea todavía no existía createForm en VnTable, por lo que la lógica de la función debería morir ahí, de esta manera no haríamos siempre, createForm Por otra parte navigate, por norma general te lleva a la card de ese módulo. La idea era usar useRoute.meta.moduleName para llevarte a la card de ese módulo. Mañana lo vemos y revisamos el alcance de la tarea porque a lo mejor hacer router.push es suficiente aunque para crear no es tan sencillo.
jsegarra closed this pull request 2025-04-15 10:34:54 +00:00
Some checks are pending
gitea/salix-front/pipeline/pr-dev Build started...
Required
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
jsegarra
Labels
Clear labels   
bug

Something is not working

  
CR / Tests passed

CR / Tests passed

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
invalid

Something is wrong

  
Missing Tests

Missing Tests

  
wontfix

This won't be fixed

No Label
bug
CR / Tests passed
duplicate
enhancement
invalid
Missing Tests
wontfix
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
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#1324
No description provided.
Delete Branch "7516-refactorNavigateAndRedirect"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?