#8363: Modified ItemFixedPrice #1561

Merged
jon merged 48 commits from 8363-RefactorItemFixedPrice into dev 2025-04-10 10:01:33 +00:00
Member
No description provided.
jon added 1 commit 2025-03-06 07:08:16 +00:00
gitea/salix-front/pipeline/pr-dev This commit is unstable Details
9f8c491eda
refactor: refs #8363 modified ItemFixedPrice and the e2e
jon added 1 commit 2025-03-07 09:51:51 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
f3fa931c70
feat: refs #8363 created composable to modify minPrice and adapt e2e
jon added 1 commit 2025-03-07 09:56:16 +00:00
jon added 1 commit 2025-03-07 12:38:03 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
22c8a8f9d6
Merge branch 'dev' into 8363-RefactorItemFixedPrice
jon added 2 commits 2025-03-11 14:59:33 +00:00
jon added 1 commit 2025-03-12 14:49:21 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
d12cd60079
refactor: refs #8363 update color when modifying an item
jon requested review from pablone 2025-03-12 15:08:18 +00:00
jon changed title from WIP: #8363: Modified ItemFixedPrice to #8363: Modified ItemFixedPrice 2025-03-12 15:08:22 +00:00
jon added 1 commit 2025-03-12 15:11:49 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
9eff4aa929
refactor: refs #8363 modified fixedPrice form test name
jon added 1 commit 2025-03-13 07:02:20 +00:00
gitea/salix-front/pipeline/pr-dev This commit is unstable Details
d376c3acbf
refactor: refs #8363 deleted useless test
jon added 1 commit 2025-03-13 07:08:18 +00:00
gitea/salix-front/pipeline/pr-dev This commit is unstable Details
db91a26114
refactor: refs #8363 deleted useless code
jon added 1 commit 2025-03-13 07:20:57 +00:00
gitea/salix-front/pipeline/pr-dev This commit is unstable Details
ad9dc4fe44
Merge branch 'dev' into 8363-RefactorItemFixedPrice
jon added 1 commit 2025-03-17 08:02:38 +00:00
gitea/salix-front/pipeline/pr-dev This commit is unstable Details
fa06a9003e
Merge branch 'dev' into 8363-RefactorItemFixedPrice
pablone requested changes 2025-03-17 08:54:38 +00:00
Dismissed
pablone left a comment
Member

Me pide que no lo revise todavía

Me pide que no lo revise todavía
jon added 3 commits 2025-03-18 14:00:56 +00:00
jon added 1 commit 2025-03-18 14:01:59 +00:00
jon added 1 commit 2025-03-20 08:16:52 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
2f5260a1a1
Merge branch 'dev' into 8363-RefactorItemFixedPrice
jon added the
CR / Tests passed
label 2025-03-21 09:58:35 +00:00
jon requested review from pablone 2025-03-21 09:58:51 +00:00
jon added 1 commit 2025-03-21 09:58:52 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
b9128cde86
Merge branch 'dev' into 8363-RefactorItemFixedPrice
pablone requested changes 2025-03-24 10:35:33 +00:00
Dismissed
@ -0,0 +1,168 @@
<script setup>
import { ref } from 'vue';
Member

Si lo pones el la carpeta de componentes ponlo con nombre genérico

Si lo pones el la carpeta de componentes ponlo con nombre genérico
jon marked this conversation as resolved
@ -0,0 +24,4 @@
const emit = defineEmits(['onDataSaved']);
const onSubmit = async () => {
if (isLoading.value) return;
Member

al clonar un articulo, solo con los campos con required * (item y warehouse ) se queda colgado i te indica que le hacen falta alguno más, veo que el create y el clone se parecen mucho yo valoraria abrir el form de create con formInitialData de la row a la que le das el clone

al clonar un articulo, solo con los campos con required `*` (item y warehouse ) se queda colgado i te indica que le hacen falta alguno más, veo que el create y el clone se parecen mucho yo valoraria abrir el form de create con `formInitialData` de la row a la que le das el clone
Member

al clonar un articulo, solo con los campos con required * (item y warehouse ) se queda colgado i te indica que le hacen falta alguno más, veo que el create y el clone se parecen mucho yo valoraria abrir el form de create con formInitialData de la row a la que le das el clone

Se te ha quedado por poner, por si te sirve de consejo, intenta utilizar el resolver conversación como un check de que ya tienes la modificación implementada

> al clonar un articulo, solo con los campos con required `*` (item y warehouse ) se queda colgado i te indica que le hacen falta alguno más, veo que el create y el clone se parecen mucho yo valoraria abrir el form de create con `formInitialData` de la row a la que le das el clone Se te ha quedado por poner, por si te sirve de consejo, intenta utilizar el resolver conversación como un check de que ya tienes la modificación implementada
jon marked this conversation as resolved
@ -0,0 +5,4 @@
import VnSelect from 'src/components/common/VnSelect.vue';
import VnInput from 'src/components/common/VnInput.vue';
import VnInputDate from 'src/components/common/VnInputDate.vue';
import VnRow from 'components/ui/VnRow.vue';
Member

Si lo pones el la carpeta de componentes ponlo con nombre genérico

Si lo pones el la carpeta de componentes ponlo con nombre genérico
jon marked this conversation as resolved
@ -0,0 +76,4 @@
data-cy="EditFixedPriceSelectOption"
@update:model-value="newValue = null"
/>
<component
Member

Cuando el component se renderiza como input se desalinean

Cuando el `component` se renderiza como input se desalinean
pablone marked this conversation as resolved
@ -74,3 +65,1 @@
optionValue: 'name',
uppercase: false,
},
component: 'select',
Member

yo pondria un input para el filtro de columna "Name" ya que así se puede aplicar un like en el back, es más cómodo para los compradores , y el select lo pondria en la columna del id

yo pondria un input para el filtro de columna "Name" ya que así se puede aplicar un like en el back, es más cómodo para los compradores , y el select lo pondria en la columna del id
pablone marked this conversation as resolved
jon added 2 commits 2025-03-24 13:03:34 +00:00
jon added 1 commit 2025-03-24 13:04:43 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
bbd771e61d
refactor: refs #8363 requested changes
jon added 1 commit 2025-03-24 15:06:10 +00:00
jon added 1 commit 2025-03-25 07:11:54 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
25bd351c17
Merge branch 'dev' into 8363-RefactorItemFixedPrice
jon requested review from pablone 2025-03-25 07:20:45 +00:00
jon added 2 commits 2025-03-25 08:18:00 +00:00
pablone reviewed 2025-03-26 08:43:36 +00:00
pablone left a comment
Member

Al cambiar el id de un articulo no se actualiza la columna name, ns si es intencional o no

Al cambiar el id de un articulo no se actualiza la columna name, ns si es intencional o no
pablone requested changes 2025-03-26 08:50:43 +00:00
Dismissed
pablone left a comment
Member

Te he vuelto a poner como no resuelto algunos comentarios de la revisión anterior

Te he vuelto a poner como no resuelto algunos comentarios de la revisión anterior
jon added 1 commit 2025-03-28 09:53:57 +00:00
jon requested review from pablone 2025-04-01 06:19:25 +00:00
jon added 1 commit 2025-04-01 07:35:15 +00:00
gitea/salix-front/pipeline/pr-dev This commit is unstable Details
7fd22cce26
Merge branch 'dev' into 8363-RefactorItemFixedPrice
jon added 1 commit 2025-04-01 10:10:08 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
6e4079de95
Merge branch 'dev' into 8363-RefactorItemFixedPrice
pablone reviewed 2025-04-01 12:52:49 +00:00
pablone left a comment
Member

Hay ocasiones en las que al modificar un articulo se produce un error y se queda pillada la app

Hay ocasiones en las que al modificar un articulo se produce un error y se queda pillada la app
pablone requested changes 2025-04-01 13:00:55 +00:00
Dismissed
pablone left a comment
Member

Revisa los comentarios de la pr

Revisa los comentarios de la pr
jon added 3 commits 2025-04-02 12:25:49 +00:00
alexm added 1 commit 2025-04-03 07:02:39 +00:00
gitea/salix-front/pipeline/pr-dev This commit is unstable Details
dc566b3902
Merge branch 'dev' into 8363-RefactorItemFixedPrice
jon added 2 commits 2025-04-04 06:23:20 +00:00
jon added 1 commit 2025-04-04 08:11:51 +00:00
gitea/salix-front/pipeline/pr-dev This commit is unstable Details
5e999e0fb1
fix: refs #8363 ended v-model
jon added 1 commit 2025-04-04 08:45:04 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
0304010f62
fix: refs #8363 skip failing e2e
jon added 1 commit 2025-04-07 11:20:06 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
11994b80b0
refactor: refs #8363 requested changes
jon added 1 commit 2025-04-07 11:23:50 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
7cc06adeef
Merge branch 'dev' into 8363-RefactorItemFixedPrice
jon requested review from pablone 2025-04-07 11:46:45 +00:00
jon added 1 commit 2025-04-07 11:47:06 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
b782da7a8d
Merge branch 'dev' into 8363-RefactorItemFixedPrice
jon added 2 commits 2025-04-09 06:50:21 +00:00
jon added 1 commit 2025-04-09 07:54:23 +00:00
gitea/salix-front/pipeline/pr-dev This commit is unstable Details
6a129c89d7
perf: refs #8363 e2e intermittence
alexm added 1 commit 2025-04-09 09:21:23 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
4351ee4976
Merge branch 'dev' into 8363-RefactorItemFixedPrice
pablone requested changes 2025-04-10 05:57:45 +00:00
Dismissed
@ -85,0 +88,4 @@
createAttrs: {
required: true,
},
width: '50px',
Member

Al final hablaste con Alex sobre las clases para los tamaños de las columnas

Al final hablaste con Alex sobre las clases para los tamaños de las columnas
Author
Member

Se ha creado un redmine para ello: https://redmine.verdnatura.es/issues/8896

Se ha creado un redmine para ello: https://redmine.verdnatura.es/issues/8896
@ -156,1 +180,3 @@
action: (row) => confirmRemove(row),
title: t('globals.clone'),
icon: 'vn:clone',
action: (row) => openCloneFixedPriceForm(row),
Member

Si le das a clonar un registro cancelas y despues le das a crear tienes los valores puestos por defecto del item al que anteriormente le habías dado a clonar

Si le das a clonar un registro cancelas y despues le das a crear tienes los valores puestos por defecto del item al que anteriormente le habías dado a clonar
jon marked this conversation as resolved
@ -371,3 +241,1 @@
data-key="ItemFixedPrices"
ref="itemFixedPriceFilterRef"
/>
<ItemFixedPriceFilter data-key="ItemFixedPrices" />
Member

Te faltan las traducciones de las tags de algunos filtros, revisa los filtros de las columnas

Te faltan las traducciones de las tags de algunos filtros, revisa los filtros de las columnas
jon marked this conversation as resolved
@ -378,2 +245,4 @@
<Teleport to="#st-data" v-if="stateStore?.isSubToolbarShown()">
<QBtnGroup push style="column-gap: 10px">
<QBtn
:disable="!hasSelectedRows"
Member

Después de editar múltiples filas y guardar a pesar de no tener seleccionado ningun registro se queda activado

Después de editar múltiples filas y guardar a pesar de no tener seleccionado ningun registro se queda activado
jon marked this conversation as resolved
@ -462,113 +293,85 @@ function handleOnDataSave({ CrudModelRef }) {
</span>
<span class="subName">{{ row.subName }}</span>
<ItemDescriptorProxy :id="row.itemFk" />
Member

pon el item descriptor proxy dentro del span sino al hacer click en la casilla te abre el descriptor

pon el item descriptor proxy dentro del span sino al hacer click en la casilla te abre el descriptor
jon marked this conversation as resolved
@ -0,0 +21,4 @@
type: String,
default: '',
},
beforeSave: {
Member

hay algunos warning al abrir el create o clone form

hay algunos warning al abrir el create o clone form
Author
Member

Los warnings se han corregido en a74e1102ed

Los warnings se han corregido en https://gitea.verdnatura.es/verdnatura/salix-front/commit/a74e1102ed6a7cdcf6bad98ed2b37312cc2fcb5f
pablone added 1 commit 2025-04-10 06:32:58 +00:00
jon added 3 commits 2025-04-10 06:56:14 +00:00
jon requested review from pablone 2025-04-10 07:57:03 +00:00
jon added 1 commit 2025-04-10 07:57:11 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
d5b5c33bc0
Merge branch 'dev' into 8363-RefactorItemFixedPrice
pablone approved these changes 2025-04-10 10:00:31 +00:00
jon merged commit d24e153033 into dev 2025-04-10 10:01:33 +00:00
jon deleted branch 8363-RefactorItemFixedPrice 2025-04-10 10:01:33 +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#1561
No description provided.