PR-3 #160

Closed
hyervoni wants to merge 0 commits from (deleted):PR-3 into dev
Member
No description provided.
hyervoni added 67 commits 2024-01-18 13:35:41 +00:00
wbuezas added 1 commit 2024-01-18 14:21:10 +00:00
alexm requested changes 2024-01-19 09:05:15 +00:00
@ -35,2 +35,3 @@
const onCityCreated = async () => {
const onCityCreated = async ({ name, provinceFk }, formData) => {
await townsFetchDataRef.value.fetch();
formData.townFk = townsLocationOptions.value.find((town) => town.name === name).id;
Member

Aqui me referiero por ejemplo https://gitea.verdnatura.es/verdnatura/salix-front/pulls/160/files#issuecomment-39480
Este no seria un enfoque correcto
Y en el caso de que se quiera hacer un filtrado por el nombre,
seria mucho mas eficiente hacer la peticion al backend pasandole el filtro que recojer todos los registros de la tabla (en produccion estas tablas tienen miles de registros).
Que como se esta haciendo ahora de recojer todo y filtrar en el frontend.

De hecho en nuestro dev (creo que lo deberiais tener en vuestra rama tambien) hemos puesto para que por ejemplo a await townsFetchDataRef.value.fetch(); le puedas pasar filter.
Ver: src/components/FetchData.vue la funcion fetch

Pero en este caso ya estamos trabajando en un enfoque mejor para el tema de codigos postales/ciudad/provincia/etc Este PR https://gitea.verdnatura.es/verdnatura/salix-front/pulls/154/files

Nos encargaremos nosotros de sustituir estos partes por el nuevo componente cuando lo tengamos

Aqui me referiero por ejemplo https://gitea.verdnatura.es/verdnatura/salix-front/pulls/160/files#issuecomment-39480 Este no seria un enfoque correcto Y en el caso de que se quiera hacer un filtrado por el nombre, seria mucho mas eficiente hacer la peticion al backend pasandole el filtro que recojer todos los registros de la tabla (en produccion estas tablas tienen miles de registros). Que como se esta haciendo ahora de recojer todo y filtrar en el frontend. De hecho en nuestro dev (creo que lo deberiais tener en vuestra rama tambien) hemos puesto para que por ejemplo a await townsFetchDataRef.value.fetch(); le puedas pasar filter. Ver: `src/components/FetchData.vue la funcion fetch` Pero en este caso ya estamos trabajando en un enfoque mejor para el tema de codigos postales/ciudad/provincia/etc Este PR https://gitea.verdnatura.es/verdnatura/salix-front/pulls/154/files Nos encargaremos nosotros de sustituir estos partes por el nuevo componente cuando lo tengamos
@ -136,2 +136,3 @@
response = await axios.patch($props.urlUpdate || $props.url, body);
}
emit('onDataSaved', formData.value);
emit('onDataSaved', formData.value, response);
Member

Aqui no he visto que se use nunca el segundo parametro dado que donde se recoja el evento se deberia poner algo como: @on-data-saved="(...args) => onDataSaved(...args)"

Pero viendo los resultados de response, igual seria mejor devolver siempre response.data
Pq ahi esta el id del nuevo registro y luego donde se usa no hace falta hacer finds por el nombre

Aqui no he visto que se use nunca el segundo parametro dado que donde se recoja el evento se deberia poner algo como: `@on-data-saved="(...args) => onDataSaved(...args)"` Pero viendo los resultados de response, igual seria mejor devolver siempre response.data Pq ahi esta el id del nuevo registro y luego donde se usa no hace falta hacer finds por el nombre
Contributor

Corregido: 7dcdaff0ef

Corregido: 7dcdaff0ef38bafc29b18b4254062c5d6b268d31
@ -0,0 +44,4 @@
return ''
}
const date = new Date(dateString || '');
return `${date.getUTCHours().toString().padStart(2, '0')}:${date.getUTCMinutes().toString().padStart(2, '0')}`;
Member

Javascript tiene una forma nativa de formatear horas:
Podria quedarse algo asi (tal cual este codigo tiene un fallo que suma una hora más pero se podrá arreglar seguro):

        new Date(dateString.value).toLocaleTimeString([], {
            hour: '2-digit',
            minute: '2-digit',
        })

info:
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Date/toLocaleTimeString#try_it
https://stackoverflow.com/questions/17913681/how-do-i-use-tolocaletimestring-without-displaying-seconds

Javascript tiene una forma nativa de formatear horas: Podria quedarse algo asi (tal cual este codigo tiene un fallo que suma una hora más pero se podrá arreglar seguro): ``` new Date(dateString.value).toLocaleTimeString([], { hour: '2-digit', minute: '2-digit', }) ``` info: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Date/toLocaleTimeString#try_it https://stackoverflow.com/questions/17913681/how-do-i-use-tolocaletimestring-without-displaying-seconds
Contributor

Corregido: 32b3d36a6a

Corregido: 32b3d36a6aefd06f1232aadd13ad8f45504be442
@ -0,0 +10,4 @@
hours = hours ? hours : 12;
let minutes = dateHour.getUTCMinutes();
minutes = minutes < 10 ? minutes.toString().padStart(2, '0') : minutes;
Member

Creo que es parecido a https://gitea.verdnatura.es/verdnatura/salix-front/pulls/160/files#issuecomment-39484
Hay formas nativas en javascript para obtener el formato 00:00

Creo que es parecido a https://gitea.verdnatura.es/verdnatura/salix-front/pulls/160/files#issuecomment-39484 Hay formas nativas en javascript para obtener el formato 00:00
Contributor

Corregido: 32b3d36a6a

Corregido: 32b3d36a6aefd06f1232aadd13ad8f45504be442
@ -1,0 +44,4 @@
],
};
const setProvince = (provinceFk) => {
Member

Pasa lo mismo de https://gitea.verdnatura.es/verdnatura/salix-front/pulls/160/files#issuecomment-39481
Lo sustituiremos nosotros por el nuevo enfoque

Pasa lo mismo de https://gitea.verdnatura.es/verdnatura/salix-front/pulls/160/files#issuecomment-39481 Lo sustituiremos nosotros por el nuevo enfoque
@ -4,0 +166,4 @@
background-color: var(--vn-light-gray);
}
}
.consignees-card-icon {
Member

Seria algo parecido a poner donde se usa:
class="ml-xs mr-sm flex items-center"

Ver: https://quasar.dev/layout/grid/introduction-to-flexbox#alignment
También por si os sirve este recurso es el que suelo usar para tener una idea de que quiero y como buscarlo en quasar:
https://somospnt.com/blog/148-flexbox-que-es-y-para-que-sirve

Seria algo parecido a poner donde se usa: `class="ml-xs mr-sm flex items-center"` Ver: https://quasar.dev/layout/grid/introduction-to-flexbox#alignment También por si os sirve este recurso es el que suelo usar para tener una idea de que quiero y como buscarlo en quasar: https://somospnt.com/blog/148-flexbox-que-es-y-para-que-sirve
First-time contributor

Corregido: 9c2267e116

Corregido: 9c2267e116
@ -3,1 +139,4 @@
</template>
<style lang="scss" scoped>
.col-content {
Member

class="p-[xs, sm, ...] "
Y para el border radius (no lo he probado) pero se puede usar algo de quasar:
https://quasar.dev/style/other-helper-classes#border-related

class="p-[xs, sm, ...] " Y para el border radius (no lo he probado) pero se puede usar algo de quasar: https://quasar.dev/style/other-helper-classes#border-related
First-time contributor

Corregido: 9d4d3bf84d

Corregido: 9d4d3bf84d
@ -1,0 +24,4 @@
const postcodesOptions = ref([]);
const onPostcodeCreated = async ({ code, provinceFk, townFk, countryFk }, formData) => {
await postcodeFetchDataRef.value.fetch();
Member

Pasa lo mismo de https://gitea.verdnatura.es/verdnatura/salix-front/pulls/160/files#issuecomment-39481
Lo sustituiremos nosotros por el nuevo enfoque

Pasa lo mismo de https://gitea.verdnatura.es/verdnatura/salix-front/pulls/160/files#issuecomment-39481 Lo sustituiremos nosotros por el nuevo enfoque
@ -3,1 +82,4 @@
</template>
<style lang="scss">
.consignees-card {
Member

Pasar a estilos de quasar si se puede, por arriba he dejado links de como se pueden hacer algunas cosas

Pasar a estilos de quasar si se puede, por arriba he dejado links de como se pueden hacer algunas cosas
First-time contributor

Corregido: 30c90e534c

Corregido: 30c90e534c
@ -40,2 +41,2 @@
const onPostcodeCreated = async () => {
postcodeFetchDataRef.value.fetch();
const onPostcodeCreated = async ({ code, provinceFk, townFk, countryFk }, formData) => {
await postcodeFetchDataRef.value.fetch();
Member

Pasa lo mismo de https://gitea.verdnatura.es/verdnatura/salix-front/pulls/160/files#issuecomment-39481
Lo sustituiremos nosotros por el nuevo enfoque

Pasa lo mismo de https://gitea.verdnatura.es/verdnatura/salix-front/pulls/160/files#issuecomment-39481 Lo sustituiremos nosotros por el nuevo enfoque
@ -0,0 +17,4 @@
const { t } = useI18n();
const route = useRoute();
const formInitialData = reactive({
Member

Esto es mas una duda.
Esto no seria lo mismo que poner:

const formInitialData = reactive({isDefaultAddress: false})
Esto es mas una duda. Esto no seria lo mismo que poner: ``` const formInitialData = reactive({isDefaultAddress: false}) ```
First-time contributor

Corregido: 2400eca1e3

Corregido: 2400eca1e3
@ -0,0 +48,4 @@
});
const onPostcodeCreated = async ({ code, provinceFk, townFk }, formData) => {
await postcodeFetchDataRef.value.fetch();
Member

Pasa lo mismo de https://gitea.verdnatura.es/verdnatura/salix-front/pulls/160/files#issuecomment-39481
Lo sustituiremos nosotros por el nuevo enfoque

Pasa lo mismo de https://gitea.verdnatura.es/verdnatura/salix-front/pulls/160/files#issuecomment-39481 Lo sustituiremos nosotros por el nuevo enfoque
@ -0,0 +35,4 @@
});
const onPostcodeCreated = async ({ code, provinceFk, townFk }, formData) => {
await postcodeFetchDataRef.value.fetch();
Member

Pasa lo mismo de https://gitea.verdnatura.es/verdnatura/salix-front/pulls/160/files#issuecomment-39481
Lo sustituiremos nosotros por el nuevo enfoque

Pasa lo mismo de https://gitea.verdnatura.es/verdnatura/salix-front/pulls/160/files#issuecomment-39481 Lo sustituiremos nosotros por el nuevo enfoque
@ -0,0 +20,4 @@
const currenciesOptions = ref([]);
</script>
<template>
<FetchData
Member

Aquí pasa parecido a lo comentado arriba. Descargarse todos los registros no es viable dado que la tabla es muy grande.
Hice código que puede valer para aquí también.
Pero igual vale la pena "incrustarlo" en el componente de VnSelectFilter, así no repetimos codigo.

Pero no hagáis aun nada un pq quiero reunirme con @juan para ver como lo podemos enfocar.
Os comento mas tarde por el grupo. Pero por dejarlo apuntado

Aquí pasa parecido a lo comentado arriba. Descargarse todos los registros no es viable dado que la tabla es muy grande. Hice código que puede valer para aquí también. Pero igual vale la pena "incrustarlo" en el componente de VnSelectFilter, así no repetimos codigo. Pero **no** hagáis aun nada un pq quiero reunirme con @juan para ver como lo podemos enfocar. Os comento mas tarde por el grupo. Pero por dejarlo apuntado
@ -0,0 +228,4 @@
componentProps: {
title: t('Confirm deletion'),
message:
rowsSelected.value.length > 1
Member

Creo que esto podría funcionar:

t(`Are you sure you want to delete this buy${rowsSelected.value.length > 1 && 's'}?`)
Creo que esto podría funcionar: ``` t(`Are you sure you want to delete this buy${rowsSelected.value.length > 1 && 's'}?`) ```
Member

Corregido, aunque tuve que usar un ternary operator para lograrlo, ya que de esa manera cuando el lenght no era mayora a 1 te concatenaba un false al string.

Commit: c7b9f4ce5b

Corregido, aunque tuve que usar un ternary operator para lograrlo, ya que de esa manera cuando el lenght no era mayora a 1 te concatenaba un `false` al string. Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/c7b9f4ce5b230989582244e09376506b74f449df
@ -0,0 +204,4 @@
</template>
<style scoped>
.list {
Member

Revisar esto, lo he visto repetido en todos los filters.
Igual sepodria definir el ancho por defencto de VnFilterPanel y asi evitar ponerlo en todos los [Section]Filter

Revisar esto, lo he visto repetido en todos los filters. Igual sepodria definir el ancho por defencto de VnFilterPanel y asi evitar ponerlo en todos los [Section]Filter
Member

Corregido.

Parecen demasiados cambios pero en realidad lo que se hizo fue mover el elemento <QList dense class="list q-gutter-y-sm q-mt-sm"> dentro del componente VnFilterPanel y quitarlo para no en todos los Filters que había, así se evita ser repetitivos con esto.

Commit: def175bee2

Corregido. Parecen demasiados cambios pero en realidad lo que se hizo fue mover el elemento `<QList dense class="list q-gutter-y-sm q-mt-sm">` dentro del componente `VnFilterPanel` y quitarlo para no en todos los Filters que había, así se evita ser repetitivos con esto. Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/def175bee25ea1d7d4112c74e9b0bb76efbd0ee0
@ -0,0 +498,4 @@
width: 100%;
}
.table-actions {
Member

Pasar a estilos de quasar

Pasar a estilos de quasar
Contributor

Corregido: e9e7cf5616

Corregido: e9e7cf56162a7fb453cc978855fbaa574c6a1758
@ -2,0 +90,4 @@
</template>
<style lang="scss" scoped>
.card-list {
Member

Aqui tambien se suele repitir estose podria o poner en el componente o igual crear una clase

Aqui tambien se suele repitir estose podria o poner en el componente o igual crear una clase
Member

Sugerencia tomada, se movieron las propiedades de card-list a una clase global y se aplico en los lugares correspondientes.

Commit: 4731bdbabd

Sugerencia tomada, se movieron las propiedades de `card-list` a una clase global y se aplico en los lugares correspondientes. Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/4731bdbabded9dc1647f178a91b47f2303f11008
jsegarra requested changes 2024-01-19 11:02:37 +00:00
@ -0,0 +102,4 @@
@on-fetch="(data) => (vehicleList = data)"
auto-load
/>
<FormModel
Member

El formulario tiene los campos en inglés

El formulario tiene los campos en inglés
Contributor

Corregido: 5bb50300ce

Corregido: 5bb50300ce6d5043bf8cd54a8d765d61f32a0474
Member

No está corregido, los campos están desordenados y aparece "Create"

No está corregido, los campos están desordenados y aparece "Create"
Contributor

Corregido: 5726ed76725eb99f070f7b874385c679c38d19f6

Corregido: 5726ed76725eb99f070f7b874385c679c38d19f6
Member

La etiqueta del campo de Fecha de creación sigue apareciendo como Created.
La posición es correcta pero el label no

La etiqueta del campo de Fecha de creación sigue apareciendo como Created. La posición es correcta pero el label no
@ -0,0 +262,4 @@
>
<template #body="{ rows }">
<div class="q-pa-md">
<QTable
Member

Los registros parecen estar ordenados por ID desc, sin embargo, la tabla no lo refleja en la cabecera

Los registros parecen estar ordenados por ID desc, sin embargo, la tabla no lo refleja en la cabecera
Contributor

Corregido: bc9ac631a5

Corregido: bc9ac631a52c4592c5215c7b4aadbc9eeaca533a
jsegarra marked this conversation as resolved
@ -0,0 +458,4 @@
</template>
<template #body-cell-actions="props">
<QTd :props="props">
<div class="table-actions">
Member

Los tooltips están en inglés

Los tooltips están en inglés
Contributor

Corregido: 9f826ebe1d

Corregido: 9f826ebe1d1e8aa1e46a870c8dee5f7c0e9096af
jsegarra marked this conversation as resolved
@ -0,0 +459,4 @@
<template #body-cell-actions="props">
<QTd :props="props">
<div class="table-actions">
<QIcon
Member

No tiene el método @click

No tiene el método @click
Contributor

Esta funcionalidad se corregira como parte del ticket: https://mindshore-verdnatura.atlassian.net/browse/SCRUM-145

Esta funcionalidad se corregira como parte del ticket: https://mindshore-verdnatura.atlassian.net/browse/SCRUM-145
jsegarra marked this conversation as resolved
@ -1 +1,192 @@
<template>Supplier consumption</template>
<script setup>
Member

El texto que aparece en la VnSearchbar está en inglés

El texto que aparece en la VnSearchbar está en inglés
Member

Corregido: 862dc20bcd

Corregido: https://gitea.verdnatura.es/verdnatura/salix-front/commit/862dc20bcd38ec8d3595a120ce23a32107af2649
jsegarra marked this conversation as resolved
jsegarra requested changes 2024-01-19 11:33:32 +00:00
@ -132,3 +142,3 @@
New postcode: Nuevo código postal
Please, ensure you put the correct data!: ¡Por favor, asegúrese de poner los datos correctos!
City: Ciudad
City: Población
Member

Se ha cambiado Ciudad por población, es correcto?
Porque en el modal de creación el titulo pone "Nuevo código de ciudad"

Se ha cambiado Ciudad por población, es correcto? Porque en el modal de creación el titulo pone "Nuevo código de ciudad"
Member

Si, a mi también me llamó la atención, lo hice de esa manera porque en el salix lo vi así, el label del Select dice "Población" y el título dice "Nueva ciudad"

Si, a mi también me llamó la atención, lo hice de esa manera porque en el salix lo vi así, el label del `Select` dice "Población" y el título dice "Nueva ciudad"
Member

Parece que en test y prod aparece Ciudad, mientras que en local pone Población.
No se que impacto tienen de cara a los usuarios, pero puede que sea error.
Propuesta, si no quita tiempo, estandarizar a Ciudad ya que hay 2 ocurrencias frente a Población.

Cualquier cosa lo comentamos.

Doy por cerrado el comentario, independientemente de lo que se haga.

Parece que en test y prod aparece Ciudad, mientras que en local pone Población. No se que impacto tienen de cara a los usuarios, pero puede que sea error. Propuesta, si no quita tiempo, estandarizar a Ciudad ya que hay 2 ocurrencias frente a Población. Cualquier cosa lo comentamos. Doy por cerrado el comentario, independientemente de lo que se haga.
jsegarra marked this conversation as resolved
@ -3,0 +73,4 @@
v-model="data.bankEntityFk"
>
<template #form>
<CreateBankEntityForm
Member

Cuando creas un nuevo Swift, en el listado de opciones no aparece. Hay que refrescar la ventana para poder usar ese valor.

Cuando creas un nuevo Swift, en el listado de opciones no aparece. Hay que refrescar la ventana para poder usar ese valor.
First-time contributor

Corregido: 23210600ac

Corregido: 23210600ac
jsegarra marked this conversation as resolved
@ -0,0 +1,202 @@
<script setup>
Member

Estando en basic-data de una entrada, pulsamos sobre el primer icono del descriptor(ficha del proveedor) y tiramos atrás muestra mensajes de error de Cannot read properties of null (reading 'emitsOptions')

Estando en basic-data de una entrada, pulsamos sobre el primer icono del descriptor(ficha del proveedor) y tiramos atrás muestra mensajes de error de Cannot read properties of null (reading 'emitsOptions')
Member

Buenas @jsegarra, al parecer es algo que también sucede en el modulo de claims, cree la siguiente issue que dice como reproducirlo y el posible problema para atacarlo en los próximos días.

Issue: https://mindshore-verdnatura.atlassian.net/browse/SCRUM-151?atlOrigin=eyJpIjoiMTc0YjUzZTkzZjA2NDFkM2ExMTY2NzNkM2EzMjMzMzMiLCJwIjoiaiJ9

Buenas @jsegarra, al parecer es algo que también sucede en el modulo de claims, cree la siguiente issue que dice como reproducirlo y el posible problema para atacarlo en los próximos días. Issue: https://mindshore-verdnatura.atlassian.net/browse/SCRUM-151?atlOrigin=eyJpIjoiMTc0YjUzZTkzZjA2NDFkM2ExMTY2NzNkM2EzMjMzMzMiLCJwIjoiaiJ9
@ -0,0 +379,4 @@
</QTable>
</template>
</VnPaginate>
<QPageSticky :offset="[20, 20]">
Member

Revisar el tooltip porque tiene un formato raro

Revisar el tooltip porque tiene un formato raro
Member

Corregido.

Commit: cae42486c8

Corregido. Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/cae42486c86b2de340e0e74b556773993bd1da39
jsegarra marked this conversation as resolved
@ -0,0 +197,4 @@
<VnRow class="row q-gutter-md q-mb-md">
<div class="col">
<QFile
label="Standard"
Member

Valor literal de Standard es correcto? En Salix no aparece esto

Valor literal de Standard es correcto? En Salix no aparece esto
Member

Corregido.

Commit: 5d0290f585

Corregido. Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/5d0290f58546ffad4ef8cd4004a37dc30d316f2f
jsegarra marked this conversation as resolved
@ -2,0 +128,4 @@
<QPageSticky :offset="[20, 20]">
<QBtn fab icon="add" color="primary" @click="redirectToCreateView()" />
<QTooltip>
{{ t('supplier.list.newSupplier') }}
Member

Creo que el texto del tooltip no coincide

Creo que el texto del tooltip no coincide
Member

Corregido

Commit: a888472ad2

Corregido Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/a888472ad226820412c3b37e6f0d52bf2a467df4
jsegarra marked this conversation as resolved
@ -1 +1,192 @@
<template>Supplier consumption</template>
<script setup>
Member

Genera un error por consola y rompe la navegación

Genera un error por consola y rompe la navegación
Member

Me di cuenta que la vista lleva un componente de filtros también.

Cree una tarea porque es un tema un poco más grande y atacarlo junto a lo que mencionas: https://mindshore-verdnatura.atlassian.net/browse/SCRUM-152?atlOrigin=eyJpIjoiMzE4ZmQzOGYwOTlkNGIyZjlmN2U0YWIzNzExYTZmYmYiLCJwIjoiaiJ9

Me di cuenta que la vista lleva un componente de filtros también. Cree una tarea porque es un tema un poco más grande y atacarlo junto a lo que mencionas: https://mindshore-verdnatura.atlassian.net/browse/SCRUM-152?atlOrigin=eyJpIjoiMzE4ZmQzOGYwOTlkNGIyZjlmN2U0YWIzNzExYTZmYmYiLCJwIjoiaiJ9
Member

Ya quedo todo implementado!

Commit: f1ff128f4a

Ya quedo todo implementado! Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/f1ff128f4aeaea51c4dadd511c37c4a2a0b2baf2
Member

No estoy de acuerdo "ya quedo todo implementado!".

Si te fijas, en Salix, el nombre del articulo está en azul y al pulsarlo te muestra una ventana. Esto en Lilium no pasa

No estoy de acuerdo "ya quedo todo implementado!". Si te fijas, en Salix, el nombre del articulo está en azul y al pulsarlo te muestra una ventana. Esto en Lilium no pasa
Member

Si, claro mil disculpas que no lo aclaré bien antes, pero habíamos decidirlo atajarlo despúes, ya que el item descriptor es un poco más complicado que los otros descriptors.

Dejo las issues relacionadas con este comment:

Si, claro mil disculpas que no lo aclaré bien antes, pero habíamos decidirlo atajarlo despúes, ya que el item descriptor es un poco más complicado que los otros descriptors. Dejo las issues relacionadas con este comment: - [Crear item descriptor](https://mindshore-verdnatura.atlassian.net/browse/SCRUM-120?atlOrigin=eyJpIjoiZDkyMzU3N2RiZWZiNDUxY2JhMjQ0NmM5MjQ2MWI0MDQiLCJwIjoiaiJ9) - [Aplicar item descriptor en lugares faltantes](https://mindshore-verdnatura.atlassian.net/browse/SCRUM-121?atlOrigin=eyJpIjoiNDE5MWE2NGMzNjg4NDQ4N2FkMzA5ODYyMDk4YzM3YmIiLCJwIjoiaiJ9)
@ -6,7 +6,6 @@ import { useI18n } from 'vue-i18n';
import { QCheckbox, QIcon } from 'quasar';
Member

Termógrafo e historial no muestra "sin resultados", es correcto?

Termógrafo e historial no muestra "sin resultados", es correcto?
Member

En esta PR todavía no están desarrollados los submodulos de travel, seguramente salgan para la próxima PR

En esta PR todavía no están desarrollados los submodulos de travel, seguramente salgan para la próxima PR
jsegarra marked this conversation as resolved
@ -290,2 +259,4 @@
</QCard>
<QCard class="full-width" v-if="entriesTableRows.length > 0">
<a class="header" :href="travelUrl + 'entry'">
{{ t('travel.summary.entries') }}
Member

Creo que algo se ha roto, porque cuando entras al detalle de un travel, y vas a la sección de Datos básicos no se muestra nada, es correcto?

Creo que algo se ha roto, porque cuando entras al detalle de un travel, y vas a la sección de Datos básicos no se muestra nada, es correcto?
Member

De momento es correcto, todavía no desarrollamos los submodulos de Travel, entrarán para el próximo PR

De momento es correcto, todavía no desarrollamos los submodulos de Travel, entrarán para el próximo PR
jsegarra marked this conversation as resolved
@ -68,3 +69,2 @@
const onPostcodeCreated = async () => {
postcodeFetchDataRef.value.fetch();
const onPostcodeCreated = async ({ code, provinceFk, townFk }, formData) => {
Member

El formulario aparece validado y con el campo de método de pago realizado

El formulario aparece validado y con el campo de método de pago realizado
Member

Corregido.

Commit: 4731bdbabd

Corregido. Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/4731bdbabded9dc1647f178a91b47f2303f11008
Member

Lo vemos, porque no veo que la solución esté bien aplicada en el commit

Lo vemos, porque no veo que la solución esté bien aplicada en el commit
Member

@jsegarra mil disculpas, pero me equivoque a la hora de dejar el commit

El commit de esta solucion es este: 7835867325

@jsegarra mil disculpas, pero me equivoque a la hora de dejar el commit El commit de esta solucion es este: https://gitea.verdnatura.es/verdnatura/salix-front/commit/78358673258863bc7e10574f4ab8bd90108c0dce
jsegarra marked this conversation as resolved
@ -357,6 +363,7 @@ onMounted(async () => {
</template>
</VnSelectCreate>
</VnRow>
<pre>{{ data }}</pre>
Member

Etiqueta pre

Etiqueta pre
Member

Corregido

Commit: 533435c7ba

Corregido Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/533435c7ba1d4587e927c39ea964055b43f27cd6
jsegarra marked this conversation as resolved
wbuezas added 1 commit 2024-01-19 13:49:19 +00:00
wbuezas added 1 commit 2024-01-19 14:16:58 +00:00
wbuezas added 1 commit 2024-01-19 14:25:31 +00:00
wbuezas added 1 commit 2024-01-19 14:30:00 +00:00
wbuezas added 1 commit 2024-01-19 14:34:20 +00:00
wbuezas added 1 commit 2024-01-19 18:16:50 +00:00
jsegarra requested changes 2024-01-22 08:13:39 +00:00
@ -1,0 +29,4 @@
},
},
],
where: { clientFk: '1' },
Member

Está puesto el clientFk en código y por tanto carga una información diferente.

Está puesto el clientFk en código y por tanto carga una información diferente.
First-time contributor

Corregido: 3fc1bff213

Corregido: 3fc1bff213
jsegarra marked this conversation as resolved
cfonseca added 1 commit 2024-01-22 11:54:42 +00:00
cfonseca added 1 commit 2024-01-22 12:02:09 +00:00
cfonseca added 1 commit 2024-01-22 12:56:08 +00:00
cfonseca added 1 commit 2024-01-22 13:44:46 +00:00
wbuezas added 2 commits 2024-01-22 15:12:30 +00:00
wbuezas added 1 commit 2024-01-22 15:26:16 +00:00
cfonseca force-pushed PR-3 from 4731bdbabd to 23210600ac 2024-01-22 16:13:52 +00:00 Compare
wbuezas added 5 commits 2024-01-22 16:16:58 +00:00
cfonseca force-pushed PR-3 from 42d41b1350 to 3fc1bff213 2024-01-22 16:29:48 +00:00 Compare
kmartinez added 1 commit 2024-01-22 16:54:45 +00:00
kmartinez added 1 commit 2024-01-22 17:06:37 +00:00
kmartinez added 1 commit 2024-01-22 17:14:14 +00:00
kmartinez added 1 commit 2024-01-22 17:27:39 +00:00
kmartinez added 1 commit 2024-01-22 19:06:12 +00:00
kmartinez added 1 commit 2024-01-23 11:16:38 +00:00
wbuezas added 4 commits 2024-01-23 13:15:57 +00:00
wbuezas added 1 commit 2024-01-23 14:43:36 +00:00
wbuezas added 1 commit 2024-01-23 14:46:45 +00:00
jsegarra requested changes 2024-01-24 11:48:42 +00:00
@ -0,0 +98,4 @@
v-close-popup
/>
<QBtn
label="Ok"
Member

Parece obvio que sea un texto tipo OK, pero mejor si seguimos el estándar

Parece obvio que sea un texto tipo OK, pero mejor si seguimos el estándar
Contributor

Corregido: 6060fab356dd60f2d34ccf2b68b9fadf37ebefae

Corregido: 6060fab356dd60f2d34ccf2b68b9fadf37ebefae
jsegarra marked this conversation as resolved
@ -1,0 +21,4 @@
onMounted(() => getBankEntities());
const getBankEntities = async () => {
Member

Esto creo que no se hace así.
He mirado en la rama dev y no me coincide con lo de esta PR

Esto creo que no se hace así. He mirado en la rama dev y no me coincide con lo de esta PR
First-time contributor

Corregido 9a23bb2ed2

Corregido 9a23bb2ed2
cfonseca force-pushed PR-3 from 6060fab356 to 9a23bb2ed2 2024-01-26 16:15:58 +00:00 Compare
Member

Fusionado y arreglado de conflictos en #166

Fusionado y arreglado de conflictos en https://gitea.verdnatura.es/verdnatura/salix-front/pulls/166
alexm closed this pull request 2024-01-29 08:34:04 +00:00

Pull request closed

Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
6 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#160
No description provided.