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

fix/OrderCatalog #893

Merged
jsegarra merged 11 commits from wbuezas/salix-front-mindshore-fork2:fix/OrderCatalog into dev 2024-11-12 12:01:43 +00:00
Conversation 0 Commits 11 Files Changed 6 +360 -196
wbuezas commented 2024-11-04 19:44:08 +00:00
Contributor
Copy Link

En esta PR se solucionan algunos problemas que se mencionaron sobre esta PR: #817

Puntos solucionados:

  • No se mantenian los tags de category, type o values
  • Formatear bien tags concatenados con and
  • Crear un dialog para cargar valores de tags similar al que existe en salix hoy en día
  • Ahora cuando se elimina una categoría se resetean los filtros correctamente
En esta PR se solucionan algunos problemas que se mencionaron sobre esta PR: [#817](https://gitea.verdnatura.es/verdnatura/salix-front/pulls/817) Puntos solucionados: - No se mantenian los tags de `category`, `type` o `values` - Formatear bien tags concatenados con `and` - Crear un dialog para cargar valores de tags similar al que existe en salix hoy en día - Ahora cuando se elimina una categoría se resetean los filtros correctamente
wbuezas added 3 commits 2024-11-04 19:44:08 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
d627c1f698 fix: order catalog
wbuezas requested review from jsegarra 2024-11-04 19:44:21 +00:00
jsegarra requested changes 2024-11-07 07:04:36 +00:00
Dismissed
src/components/ui/VnFilterPanel.vue
@ -170,9 +169,29 @@ const tagsList = computed(() => {
return tagList;
});
const formatTags = (tags) => {
jsegarra commented 2024-11-07 06:57:24 +00:00
Member
Copy Link

La IA me ha propiesto esta solucion
const formatTags = (tags) => { return tags.flatMap((tag) => tag.label === 'and' ? tag.value.flatMap(item => Object.entries(item).map(([key, value]) => ({ label: key, value })) ) : [tag] ); };

Con reduce es mas complejo

La IA me ha propiesto esta solucion `const formatTags = (tags) => { return tags.flatMap((tag) => tag.label === 'and' ? tag.value.flatMap(item => Object.entries(item).map(([key, value]) => ({ label: key, value })) ) : [tag] ); };` Con reduce es mas complejo
wbuezas commented 2024-11-08 11:57:37 +00:00
Author
Contributor
Copy Link

Decidí no aplicar la sugerencia ya que considero que complejiza y hace el codigo más dificil de leer / mantener.

Decidí no aplicar la sugerencia ya que considero que complejiza y hace el codigo más dificil de leer / mantener.
👍 1
jsegarra marked this conversation as resolved
src/pages/Order/Card/CatalogFilterValueDialog.vue Outdated
@ -0,0 +66,4 @@
<VnSelect
:label="t('params.tag')"
v-model="selectedTag"
:options="props.tags || []"
jsegarra commented 2024-11-07 06:55:30 +00:00
Member
Copy Link

porque pones un valor por defecto si ya tiene el default?

porque pones un valor por defecto si ya tiene el default?
wbuezas commented 2024-11-07 22:07:53 +00:00
Author
Contributor
Copy Link

Buena observación, había quedado de antes que componentice esta parte.

Commit: 498a52a3e5

Buena observación, había quedado de antes que componentice esta parte. Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/498a52a3e5a78c4e8c8f7c58a3cc4b655ca33cac
jsegarra marked this conversation as resolved
src/pages/Order/Card/OrderCatalogFilter.vue
@ -217,0 +193,4 @@
: removeTagGroupParam(params, searchFn, valIndex)
"
>
<strong v-if="customTag.label === 'categoryFk'">
jsegarra commented 2024-11-07 07:04:17 +00:00
Member
Copy Link

ufff 2 veces la misma comprobacion...

ufff 2 veces la misma comprobacion...
👀 1
jsegarra marked this conversation as resolved
src/pages/Order/Card/OrderCatalogFilter.vue Outdated
@ -385,2 +321,4 @@
</VnInput>
</QItem>
<QSeparator />
<QDialog v-model="showValueFilterDialog">
jsegarra commented 2024-11-07 06:58:34 +00:00
Member
Copy Link

Prueba con QPopupProxy mejor que dialogo porque así se abre en la misma región y no hay que mover tanto el ratón y vista.
También nos facilita copiar y pegar valores

Prueba con QPopupProxy mejor que dialogo porque así se abre en la misma región y no hay que mover tanto el ratón y vista. También nos facilita copiar y pegar valores
wbuezas commented 2024-11-07 22:16:42 +00:00
Author
Contributor
Copy Link

Coincido, QPopupProxy aplicado.

Commit: 52a2250acc

Coincido, `QPopupProxy` aplicado. Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/52a2250acc721f80a9a5e5d85dd4b9bad99ea1df
jsegarra marked this conversation as resolved
wbuezas added 1 commit 2024-11-07 22:07:14 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
498a52a3e5 refactor: small change
wbuezas added 1 commit 2024-11-07 22:16:15 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
52a2250acc refactor: apply QPopupProxy
wbuezas added 1 commit 2024-11-08 00:10:16 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
8bff7fc1d6 fix: reset category
jsegarra added 1 commit 2024-11-08 06:59:29 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
96d68ed106 Merge branch 'dev' into fix/OrderCatalog
jsegarra added 1 commit 2024-11-08 12:11:53 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
3825dc3851 feat: disabled buttons
wbuezas added 1 commit 2024-11-08 13:11:25 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
c1d623622d refactor: remove unused variable
wbuezas added 1 commit 2024-11-08 13:13:46 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
ade90afd03 refactor: change keyup.enter for update:model-value
jsegarra added 1 commit 2024-11-12 11:30:38 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
a7d53fcbe6 Merge branch 'dev' into fix/OrderCatalog
jsegarra approved these changes 2024-11-12 11:30:43 +00:00
jsegarra merged commit 830c3658d4 into dev 2024-11-12 12:01:43 +00:00
jsegarra deleted branch fix/OrderCatalog 2024-11-12 12:01:43 +00:00
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#893
No description provided.
Delete Branch "wbuezas/salix-front-mindshore-fork2:fix/OrderCatalog"

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?