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

EntryLatestBuysFilters #226

Merged
jsegarra merged 12 commits from :EntryLatestBuysFilters into dev 2024-03-08 12:08:57 +00:00
Conversation 2 Commits 12 Files Changed 3 +918 -31
hyervoni commented 2024-03-05 11:32:46 +00:00
Contributor
Copy Link

https://redmine.verdnatura.es/issues/6537

https://redmine.verdnatura.es/issues/6537
hyervoni added 6 commits 2024-03-05 11:32:46 +00:00
hyervoni requested review from alexm 2024-03-05 11:33:02 +00:00
hyervoni requested review from jsegarra 2024-03-05 11:33:02 +00:00
jsegarra commented 2024-03-06 09:26:43 +00:00
Member
Copy Link

Diría que hay 2 errores en el dialogo cuando editas las filas

  1. El número de filas que se están editando, tiene un formato diferente a Salix.
  2. Los botones no están en el mismo orden
Diría que hay 2 errores en el dialogo cuando editas las filas 1. El número de filas que se están editando, tiene un formato diferente a Salix. 2. Los botones no están en el mismo orden
jsegarra requested changes 2024-03-06 09:26:46 +00:00
src/pages/Entry/EntryLatestBuys.vue
@ -29,0 +87,4 @@
return col.columnFilter.type === 'select'
? { 'update:modelValue': () => applyColumnFilter(col) }
: {
'keyup.enter': () => applyColumnFilter(col),
jsegarra commented 2024-03-06 09:10:53 +00:00
Member
Copy Link

Probando, diría que sería mas usable (UX) añadir el evento de tab.
He probado añadiendo keydown.tab.prevent o keydown.enter.tab , y no me ha ido

Probando, diría que sería mas usable (UX) añadir el evento de tab. He probado añadiendo keydown.tab.prevent o keydown.enter.tab , y no me ha ido
wbuezas commented 2024-03-06 20:59:56 +00:00
Contributor
Copy Link

Mmmm.. Probe un poco y tampoco me funciono

Mmmm.. Probe un poco y tampoco me funciono
jsegarra commented 2024-03-07 06:22:18 +00:00
Member
Copy Link

Como no se encuentra en salix, lo dejamos como está. lo apuntamos para nuestro lado.

Como **no se encuentra en salix**, **lo dejamos como está**. lo apuntamos para nuestro lado.
src/pages/Entry/EntryLatestBuys.vue Outdated
@ -29,1 +91,4 @@
};
};
const columns = computed(() => [
jsegarra commented 2024-03-06 09:04:59 +00:00
Member
Copy Link

Siguiendo en la línea de otras tareas, añadiría clearable a los inputs. Lo he probado, pero clearable:true no limpia el input
Otro enfoque es el que hemos seguido aqui #198

Siguiendo en la línea de otras tareas, añadiría clearable a los inputs. Lo he probado, pero clearable:true no limpia el input Otro enfoque es el que hemos seguido aqui https://gitea.verdnatura.es/verdnatura/salix-front/pulls/198
wbuezas commented 2024-03-06 22:09:55 +00:00
Contributor
Copy Link

Como esto no se encuentra en salix, si realmente queremos agregarlo, te parece que lo saquemos en una PR aparte como una feature aparte?

Como esto no se encuentra en salix, si realmente queremos agregarlo, te parece que lo saquemos en una PR aparte como una feature aparte?
jsegarra commented 2024-03-07 06:22:04 +00:00
Member
Copy Link

Como no se encuentra en salix, lo dejamos como está. lo apuntamos para nuestro lado.

Como **no se encuentra en salix**, **lo dejamos como está**. lo apuntamos para nuestro lado.
jsegarra commented 2024-03-07 08:24:36 +00:00
Member
Copy Link

Si que hay una solución a este caso, que es mergear la rama #198 en esta y modificar getInputEvents para que 'update:modelValue' no sea exclusivo del tipo select.

Quizás podríamos ver si podemos evitar duplicar la línea de update:modelValue

Te lo dejo como TIP, sin embargo lo vamos a abordar en la tarea #6998

Si que hay una solución a este caso, que es mergear la rama https://gitea.verdnatura.es/verdnatura/salix-front/pulls/198 en esta y modificar getInputEvents para que 'update:modelValue' no sea exclusivo del tipo select. Quizás podríamos ver si podemos evitar duplicar la línea de update:modelValue Te lo dejo como TIP, sin embargo lo vamos a abordar en la tarea #6998
src/pages/Entry/EntryLatestBuys.vue Outdated
@ -290,3 +700,3 @@
<QTd @click.stop>
<QBtn flat color="blue">
<QBtn flat color="primary">
{{ row.itemFk }}
jsegarra commented 2024-03-06 09:26:15 +00:00
Member
Copy Link

🚩 es un boton pero no hace nada.
Creo que le falta

🚩 es un boton pero no hace nada. Creo que le falta <ItemDescriptorProxy :id="row.itemFk" />
wbuezas commented 2024-03-06 21:24:23 +00:00
Contributor
Copy Link

Si, llevaba un descriptor, se me habra pasado, ahi lo agregue

Commit: 44d7f98cdc

Si, llevaba un descriptor, se me habra pasado, ahi lo agregue Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/44d7f98cdcc9b15eb530776f99d8e78556feb800
😆 1
jsegarra marked this conversation as resolved
wbuezas added 4 commits 2024-03-06 21:22:24 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
44d7f98cdc Add item descriptor
wbuezas added 1 commit 2024-03-06 22:01:00 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
2449488927 Changes
wbuezas commented 2024-03-06 22:03:53 +00:00
Contributor
Copy Link

Diría que hay 2 errores en el dialogo cuando editas las filas

  1. El número de filas que se están editando, tiene un formato diferente a Salix.
  2. Los botones no están en el mismo orden
  1. Formato de salix aplicado, lo unico que no aplique igual es el color "rojo anaranjado" ya que no existe actualmente en lilium, asi que le coloque el color primary, si se quiere traer ese color especifico a lilium avisenme y lo hago
  2. Orden corregido

Commit: 2449488927

> Diría que hay 2 errores en el dialogo cuando editas las filas > 1. El número de filas que se están editando, tiene un formato diferente a Salix. > 2. Los botones no están en el mismo orden > 1. Formato de salix aplicado, lo unico que no aplique igual es el color "rojo anaranjado" ya que no existe actualmente en lilium, asi que le coloque el color `primary`, si se quiere traer ese color especifico a lilium avisenme y lo hago 2. Orden corregido Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/2449488927837d247432614b79bd948b1e358732
👍 1
jsegarra approved these changes 2024-03-07 06:23:10 +00:00
alexm approved these changes 2024-03-08 06:52:59 +00:00
wbuezas added 1 commit 2024-03-08 12:07:26 +00:00
jsegarra merged commit 6baa158d43 into dev 2024-03-08 12:08:57 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
jsegarra
alexm
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
4 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#226
No description provided.
Delete Branch ":EntryLatestBuysFilters"

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?