#6896 end orders migration #400

Merged
jon merged 17 commits from 6896-EndOrderMigration into dev 2024-06-20 12:31:45 +00:00
Member
No description provided.
jon added 1 commit 2024-05-23 11:56:49 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
98159de257
refactor: refs #6896 end migration orders
jon added 1 commit 2024-05-23 11:59:16 +00:00
jon added 1 commit 2024-05-23 12:06:04 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
6e59f3ae12
refactor: refs #6896 fix qdrawer
jon changed title from WIP: #6896 end migration orders to WIP: #6896 end orders migration 2024-06-03 09:36:49 +00:00
jon added 1 commit 2024-06-03 09:44:04 +00:00
jon added 2 commits 2024-06-04 07:54:53 +00:00
jon added 1 commit 2024-06-17 07:56:25 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
42008ef74b
feat: refs #6896 created VnImg and added to order module
jon changed title from WIP: #6896 end orders migration to #6896 end orders migration 2024-06-17 07:57:13 +00:00
jon added 1 commit 2024-06-17 07:57:18 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
9c1e79ebee
Merge branch 'dev' into 6896-EndOrderMigration
jon added 2 commits 2024-06-17 07:58:29 +00:00
jon requested review from jgallego 2024-06-17 07:59:19 +00:00
jgallego requested review from jsegarra 2024-06-17 10:43:30 +00:00
jgallego requested changes 2024-06-17 10:44:34 +00:00
Dismissed
@ -72,1 +72,3 @@
<QImg :src="image" spinner-color="primary" style="min-height: 256px">
<VnImg
:src="image"
spinner-color="primary"
Owner

estos no pueden ir en el mismo componente?

estos no pueden ir en el mismo componente?
jon marked this conversation as resolved
@ -258,0 +326,4 @@
justify-content: center;
text-align: left;
height: auto;
padding-top: 12px;
Owner

hay mucho estilo, en general no hay que usar estilo, sino que se use lo nativo, pongo a @jsegarra para que lo valide y/o te explique como implementarlo.

hay mucho estilo, en general no hay que usar estilo, sino que se use lo nativo, pongo a @jsegarra para que lo valide y/o te explique como implementarlo.
jon marked this conversation as resolved
jsegarra reviewed 2024-06-17 11:03:02 +00:00
@ -73,0 +72,4 @@
<VnImg
:src="image"
spinner-color="primary"
style="min-height: 256px"
Member

De echo, si pones esta prop modificas la imagen y no se ve entera.
Por ejemplo, http://localhost:9000/#/item/6/summary?limit=10

  1. Con el style, la imagen se corta en las letras, cuando realmente no es así
  2. Spinner-color no es una prop del component VnImg, asi que no haria falta definirla
  3. Te propondria definir un valor por defecto a la prop de zoomSize,por ejemplo "lg"
De echo, si pones esta prop modificas la imagen y no se ve entera. Por ejemplo, http://localhost:9000/#/item/6/summary?limit=10 1. Con el style, la imagen se corta en las letras, cuando realmente no es así 2. Spinner-color no es una prop del component VnImg, asi que no haria falta definirla 3. Te propondria definir un valor por defecto a la prop de zoomSize,por ejemplo "lg"
Member

Veo que has puesto v-bind="$attrs, guay.
Sin embargo, con respecto a spinner-color, si lo pones en este component, todas las instancias usaran este color y no el que hayas puesto en las instancias

Veo que has puesto v-bind="$attrs, guay. Sin embargo, con respecto a spinner-color, si lo pones en este component, todas las instancias usaran este color y no el que hayas puesto en las instancias
jon marked this conversation as resolved
jsegarra requested changes 2024-06-17 11:12:21 +00:00
Dismissed
@ -0,0 +10,4 @@
type: Boolean,
default: false,
},
zoomSize: {
Member

La defines como String pero el valor no lo usas en ningún lado.
En la 32, compruebas si está o no

La defines como String pero el valor no lo usas en ningún lado. En la 32, compruebas si está o no
jsegarra marked this conversation as resolved
@ -0,0 +46,4 @@
border-radius: 0%;
}
.img_zoom {
width: 12%;
Member

Aquí es donde modificando el valor, consigues mas o menos zoom

Aquí es donde modificando el valor, consigues mas o menos zoom
Member

Te diria de mirarte la propiedad v-bind ya que puedes vincular propiedades al apartado de style

Te diria de mirarte la propiedad v-bind ya que puedes vincular propiedades al apartado de style
Member
Te paso un ejemplo https://play.vuejs.org/#eNp9ks1u2zAQhF9lyx6UBInVoD25ioHWCNAWRVvUBXrhhaHWMmOKJPjj2BD07llScRIgQXTS7oyW36w4sC/OzXYJ2Zw1QXrlIgSMyS24Ub2zPsIAHtcwwtrbHiqyVtxwI60JEfrQwVXWT6pvqLWF/9br9l11ejRIq60/Wt5/KE91+hnqGpZFanEtko6wEzohN009QdDxVETsnRYRqQJoNpcgtQjhirM7L5xDz9liGArEODb15nLyKeMSDbzobYuazIWBM6hJbupnM5sQDxohSOuwpXr2MBaGPOdGyG3nbTLtRZkwp5E3yrQnpcoZzuCnMtuHjJQQPRqJEC3EDb74HparFThPZ/l4ILuQUe1QH+Cs5mYs0TPOgp2zSEhmrbrZbbCG/kzhyUF6pzT63y4q2i5n84k0a4K2f/ej9KJPeH7syw3K7Sv927DPPc7+eAzod8jZoxaF7zBO8vXqF+7p/VGkrSZN7jfEvxisTplxsn2lHRD2M1+h/V7ulzLdv3C9j2jCMVQGzc6x+DmjO7d8I/oT7sfZp/Id7ZON92vW9Yo=
jon marked this conversation as resolved
jsegarra requested changes 2024-06-17 11:13:22 +00:00
Dismissed
@ -0,0 +42,4 @@
<style lang="scss" scoped>
.img {
cursor: pointer;
Member

La acción es de zoom, así que pondría zoom-in

La acción es de zoom, así que pondría zoom-in
jsegarra marked this conversation as resolved
jsegarra requested changes 2024-06-17 11:47:26 +00:00
Dismissed
@ -160,2 +141,2 @@
params.orderBy = JSON.stringify(orderBy);
search();
const onOrderFieldChange = (value, params) => {
console.log('value: ', value);
Member

💣👀

💣👀
jon marked this conversation as resolved
@ -162,0 +142,4 @@
console.log('value: ', value);
console.log('params: ', params);
const tagObj = JSON.parse(params.orderBy);
switch (value) {
Member

En este switch se repite la la asignación de params.orderBy y los console.logs

Se puede reducir definiendo un objeto en el que el "case" sea la clave y la asignación de tagObj.field el valor

En este switch se repite la la asignación de params.orderBy y los console.logs Se puede reducir definiendo un objeto en el que el "case" sea la clave y la asignación de tagObj.field el valor
jon marked this conversation as resolved
@ -188,3 +211,4 @@
@remove="clearFilter"
>
<template #tags="{ tag, formatFn }">
{{ console.log('tag: ', tag) }}
Member

💣👀

💣👀
jon marked this conversation as resolved
@ -30,3 +31,3 @@
<QCard class="card shadow-6">
<div class="img-wrapper">
<QImg
<VnImg
Member

Propuesta: dejar src y class. Si quieres darle otro estilo, lo comentamos

Propuesta: dejar src y class. Si quieres darle otro estilo, lo comentamos
jon marked this conversation as resolved
@ -32,3 +33,3 @@
<QImg
<VnImg
:src="`/api/Images/catalog/200x200/${item.id}/download?access_token=${token}`"
spinner-color="primary"
Member

Si le pongo red, se sigue viendo naranja

Si le pongo red, se sigue viendo naranja
jon marked this conversation as resolved
@ -33,3 +33,4 @@
<VnImg
:src="`/api/Images/catalog/200x200/${item.id}/download?access_token=${token}`"
spinner-color="primary"
:ratio="1"
Member

Si le pongo 10, sigue aplicando 1

Si le pongo 10, sigue aplicando 1
jon marked this conversation as resolved
@ -44,0 +37,4 @@
const client = clientList.value.find(
(client) => client.id === initialFormState.clientFk
);
if (client && client.defaultAddressFk) {
Member

podemos simplificarlo

if (!client?.defaultAddressFk)
        throw new Error(t(`No default address found for the client`));
    fetchAddressList(client.defaultAddressFk);
podemos simplificarlo ``` if (!client?.defaultAddressFk) throw new Error(t(`No default address found for the client`)); fetchAddressList(client.defaultAddressFk); ```
jon marked this conversation as resolved
@ -135,3 +149,3 @@
<FormModel
:url="!isNew ? `Orders/${route.params.id}` : null"
:url-create="isNew ? 'Orders/new' : null"
:url-create="'Orders/new'"
Member

url-create="Orders/new"

url-create="Orders/new"
jon marked this conversation as resolved
@ -43,6 +44,7 @@ function confirmRemove(item) {
}
async function remove(item) {
console.log('item: ', item);
Member

💣👀

💣👀
jon marked this conversation as resolved
@ -133,2 +162,2 @@
:title="row?.item?.name"
class="cursor-inherit"
<div class="q-pa-md">
<QTable
Member

Hemos cambiado CardList por QTable?? esta no es la manera correcta, si quieres vemos como definir bien las tablas.

Aunque te digo de dejarlo para el final porque se viene VnTable y a lo mejor cambia la cosa. O como tu veas

Hemos cambiado CardList por QTable?? esta no es la manera correcta, si quieres vemos como definir bien las tablas. Aunque te digo de dejarlo para el final porque se viene VnTable y a lo mejor cambia la cosa. O como tu veas
jon marked this conversation as resolved
@ -255,6 +319,38 @@ async function confirmOrder() {
color: var(--vn-label-color);
text-align: center;
}
Member

Mmm...has definido una css class para cada columna?
Lo vemos con detalle 1 a 1

Mmm...has definido una css class para cada columna? Lo vemos con detalle 1 a 1
jon marked this conversation as resolved
jsegarra requested changes 2024-06-17 11:53:49 +00:00
Dismissed
@ -0,0 +1,52 @@
<script setup>
import { ref } from 'vue';
const $props = defineProps({
Member

Lo ideal de este componente seria montar la url en este componente pasandole la coleccion(user/Images, etc), el tamaño(200x200, 50x50, etc), el id.
Así podemos reemplazar QImg y la propiedad src por VnImg y las props de antes

Lo ideal de este componente seria montar la url en este componente pasandole la coleccion(user/Images, etc), el tamaño(200x200, 50x50, etc), el id. Así podemos reemplazar QImg y la propiedad src por VnImg y las props de antes
jon marked this conversation as resolved
@ -0,0 +2,4 @@
import { ref } from 'vue';
const $props = defineProps({
collection: {
Member

Esta prop no se usa para nada, no?

Esta prop no se usa para nada, no?
jon marked this conversation as resolved
@ -0,0 +14,4 @@
type: String,
required: true,
},
id: {
Member

Esta tampoco

Esta tampoco
jon marked this conversation as resolved
jon added 1 commit 2024-06-18 08:46:21 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
a5f4f85c80
refactor: refs #6896 requested changes
jon requested review from jsegarra 2024-06-18 08:47:20 +00:00
jon added 1 commit 2024-06-18 11:31:31 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
3023280108
refactor: refs #6896 refactor VnImg
jon added 1 commit 2024-06-18 11:32:16 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
db67fd1e43
Merge branch 'dev' into 6896-EndOrderMigration
jsegarra added 1 commit 2024-06-19 11:43:39 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
3fdad15f21
fix: comments
jsegarra requested changes 2024-06-19 11:43:46 +00:00
Dismissed
@ -3,4 +3,11 @@ import qFormMixin from './qformMixin';
export default boot(({ app }) => {
Member

eliminar
Ya no sirve

eliminar Ya no sirve
jon marked this conversation as resolved
@ -193,1 +216,3 @@
/>
<template #body-cell-description="{ value }">
<QTd class="description-cell">
<FetchedTags
Member

Nos e muestran porque te falta pasarle row.item

Nos e muestran porque te falta pasarle row.item
jon marked this conversation as resolved
@ -194,2 +229,3 @@
</QTd>
</template>
</CardList>
<template #body-cell-warehouse="{ value }">
Member

Eliminar body-cell-XXX cuando sólo se muestra valor

Eliminar body-cell-XXX cuando sólo se muestra valor
jon marked this conversation as resolved
@ -196,0 +261,4 @@
size="sm"
class="cursor-pointer"
@click.stop="confirmRemove(value)"
style="margin-left: 40%"
Member

es necesario?

es necesario?
jon marked this conversation as resolved
@ -257,1 +325,4 @@
}
.description-cell {
width: 25%;
Member

poniendo 5, 10, 15, 20 o 25 tenemos el mismo resultado

poniendo 5, 10, 15, 20 o 25 tenemos el mismo resultado
jon marked this conversation as resolved
@ -0,0 +1,12 @@
import { defineStore } from 'pinia';
Member

Eliminar fichero ya que no se está usando

Eliminar fichero ya que no se está usando
jon marked this conversation as resolved
jon added 1 commit 2024-06-20 06:33:37 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
a84c20f2ce
refactor: refs #6896 changes requested in PR
jon requested review from jsegarra 2024-06-20 06:34:24 +00:00
jsegarra approved these changes 2024-06-20 08:33:46 +00:00
jon requested review from jgallego 2024-06-20 08:39:23 +00:00
jon added 1 commit 2024-06-20 08:39:34 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
a61196b04d
Merge branch 'dev' into 6896-EndOrderMigration
jon added 1 commit 2024-06-20 10:12:25 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
304c7677bc
Merge branch 'dev' into 6896-EndOrderMigration
jgallego approved these changes 2024-06-20 12:31:15 +00:00
jon merged commit b60fa62c99 into dev 2024-06-20 12:31:45 +00:00
jon deleted branch 6896-EndOrderMigration 2024-06-20 12:31:45 +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#400
No description provided.