Desarrollo de los submodulos basic data, notes y pbx #245

Merged
jsegarra merged 16 commits from :features/ms_submodulos_worker into dev 2024-04-08 06:19:29 +00:00
Contributor
https://redmine.verdnatura.es/issues/6551
hyervoni added 1 commit 2024-03-18 13:42:40 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
844cd0e12b
Desarrollo de los submodulos basic data, notes y pbx
cfonseca added 1 commit 2024-03-18 13:50:08 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
5867649d0c
Merge branch 'dev' into features/ms_submodulos_worker
jsegarra requested changes 2024-03-18 14:29:56 +00:00
Dismissed
@ -0,0 +46,4 @@
};
</script>
<template>
Member

El campo no tiene focus cuando se crea el formulario

El campo no tiene focus cuando se crea el formulario
First-time contributor

Corregido: c06d810fe9

Corregido: c06d810fe9
jsegarra marked this conversation as resolved
jsegarra requested changes 2024-03-18 14:31:21 +00:00
Dismissed
@ -0,0 +21,4 @@
const workersRef = ref(null);
const id = ref(route.params.id);
const hasChanged = computed(() => {
Member

Cuando se modifica no actualiza el valor del Descriptor

Cuando se modifica no actualiza el valor del Descriptor
First-time contributor

Corregido: c06d810fe9

Corregido: c06d810fe9
jsegarra marked this conversation as resolved
jsegarra requested changes 2024-03-18 14:35:49 +00:00
Dismissed
@ -0,0 +79,4 @@
<QForm>
<VnRow class="row q-gutter-md q-mb-md">
<div class="col">
<QInput :label="t('Note')" type="textarea" v-model="text" />
Member

Duda UI/UX, cuando pego un texto tipo "Lorem Ipsum", tengo que hacer mucho scroll para ver el texto, mientras que tengo un espacio bien amplio por los lados.

Es correcto esto?
Es Salix no pasa

Duda UI/UX, cuando pego un texto tipo "Lorem Ipsum", tengo que hacer mucho scroll para ver el texto, mientras que tengo un espacio bien amplio por los lados. Es correcto esto? Es Salix no pasa
First-time contributor

Corregido: c06d810fe9

Corregido: c06d810fe9
Member

Hemos visto que en Salix está así, por ello no es error, sino propuesta de mejora para ocupar el ancho de la ventana

Hemos visto que en Salix está así, por ello no es error, sino propuesta de mejora para ocupar el ancho de la ventana
First-time contributor

Corregido: 1d37b91e09

Corregido: 1d37b91e09
jsegarra marked this conversation as resolved
cfonseca added 1 commit 2024-03-20 16:53:22 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
c06d810fe9
Soluciones a comentarios 1
cfonseca added 1 commit 2024-03-20 16:55:12 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
142155e14e
Merge branch 'dev' into features/ms_submodulos_worker
jsegarra requested changes 2024-03-21 06:20:53 +00:00
Dismissed
@ -0,0 +105,4 @@
/>
<Teleport to="#st-actions">
<QBtnGroup push class="q-gutter-x-sm">
Member

El botón de restarurar no está funcionando o no está habilitado cuando modifico el formulario

El botón de restarurar no está funcionando o no está habilitado cuando modifico el formulario
First-time contributor

Corregido: 1d37b91e09

Corregido: 1d37b91e09
jsegarra marked this conversation as resolved
@ -113,3 +115,3 @@
</template>
</VnLv>
<VnLv :value="sip">
<VnLv :value="state.get('extension') || sip">
Member

El valor se está mostrando pero el icono no

El valor se está mostrando pero el icono no
First-time contributor

Corregido: 1d37b91e09

Corregido: 1d37b91e09
jsegarra marked this conversation as resolved
jsegarra requested review from alexm 2024-03-21 06:22:41 +00:00
alexm requested changes 2024-03-22 06:42:34 +00:00
Dismissed
@ -0,0 +48,4 @@
}
);
const getData = async (id) => {
Member

Usar componente FormModel

Usar componente FormModel
First-time contributor

Corregido: 1d37b91e09, se implementa el FormModel en el componente como se solicita, encontré unos puntos de mejora que subiré en el próximo commit.

Corregido: 1d37b91e09, se implementa el FormModel en el componente como se solicita, encontré unos puntos de mejora que subiré en el próximo commit.
jsegarra marked this conversation as resolved
@ -0,0 +29,4 @@
}
);
const getData = async (id) => {
Member

Usar componente VnNotes (src/components/ui/VnNotes.vue)

Usar componente VnNotes (src/components/ui/VnNotes.vue)
First-time contributor

Corregido: 8cf14929e3

Corregido: 8cf14929e3
Member

Se muestran los segundos pero es tema del composable toDateHour
Por otra parte, si relleno el input pero pulso sobre la X, no muestra el modal de confirmar salida. Esto creo que es punto de mejora en VnNotes. tu que dices @alexm ??

Se muestran los segundos pero es tema del composable toDateHour Por otra parte, si relleno el input pero pulso sobre la X, no muestra el modal de confirmar salida. Esto creo que es punto de mejora en VnNotes. tu que dices @alexm ??
jsegarra marked this conversation as resolved
@ -0,0 +87,4 @@
};
</script>
<template>
Member

Usar componente FormModel

Usar componente FormModel
First-time contributor

Corregido: 081d4bf5fc

Corregido: 081d4bf5fc
jsegarra marked this conversation as resolved
jsegarra started working 2024-03-22 09:17:08 +00:00
jsegarra stopped working 2024-03-22 09:17:15 +00:00
7 seconds
cfonseca added 1 commit 2024-03-26 22:18:05 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
1d37b91e09
Solucion a comentarios 3
cfonseca added 1 commit 2024-03-27 04:12:19 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
081d4bf5fc
Solucion a comentarios 4
jsegarra requested changes 2024-03-27 07:44:02 +00:00
Dismissed
@ -0,0 +33,4 @@
</script>
<template>
<FormModel
Member

Los botones no se habilitan cuando se modifica el campo

Los botones no se habilitan cuando se modifica el campo
Contributor

Corregido

Commit: 239daceca6

Corregido Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/239daceca60bd3158ea8faa79c851a137a7e3a4a
jsegarra marked this conversation as resolved
@ -0,0 +36,4 @@
<FormModel
:filter="filter"
:form-initial-data="formInitialData"
:url="`Workers/${route.params.id}`"
Member

En el caso de que este funcionando bien. Al pulsar guardar, que URL se usaría

En el caso de que este funcionando bien. Al pulsar guardar, que URL se usaría
Contributor

Corregido

Commit: 239daceca6

Corregido Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/239daceca60bd3158ea8faa79c851a137a7e3a4a
jsegarra marked this conversation as resolved
@ -0,0 +44,4 @@
<template #form="{ data }">
<VnRow class="row q-gutter-md q-mb-md">
<div class="col">
<QInput
Member

QInput? Usar VnInput

QInput? Usar VnInput
Contributor

Corregido

Commit: 239daceca6

Corregido Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/239daceca60bd3158ea8faa79c851a137a7e3a4a
jsegarra marked this conversation as resolved
@ -0,0 +45,4 @@
<VnRow class="row q-gutter-md q-mb-md">
<div class="col">
<QInput
:label="t('worker.summary.sipExtension')"
Member

Cuando se pierde el foco del campo el valor del input desaparece

Cuando se pierde el foco del campo el valor del input desaparece
Contributor

Corregido

Commit: 239daceca6

Corregido Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/239daceca60bd3158ea8faa79c851a137a7e3a4a
jsegarra marked this conversation as resolved
cfonseca added 1 commit 2024-03-27 13:52:50 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
8cf14929e3
Se implementa nuevo componente para la gestion de notas
wbuezas added 1 commit 2024-03-28 16:45:46 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
239daceca6
Apply changes in Worker PBX
jsegarra requested changes 2024-03-28 22:34:28 +00:00
Dismissed
@ -0,0 +1,192 @@
<script setup>
import { reactive, ref } from 'vue';
Member

reactive no está en uso

reactive no está en uso
Contributor

Import de reactive eliminado

Commit: 507c979cc2

Import de `reactive` eliminado Commit: https://gitea.verdnatura.es/hyervoni/salix-front-mindshore/commit/507c979cc2d6219a4f2d5e55b21ec6a243b25076
jsegarra marked this conversation as resolved
jsegarra requested review from alexm 2024-03-28 23:35:43 +00:00
alexm requested changes 2024-04-02 06:34:04 +00:00
Dismissed
@ -0,0 +47,4 @@
];
</script>
<template>
Member

FechData

FechData
Contributor

Nombre actualizado

Commit: 3569afdb85

Nombre actualizado Commit: https://gitea.verdnatura.es/hyervoni/salix-front-mindshore/commit/3569afdb85f612696e4c36404402ae92fc460d68
jsegarra marked this conversation as resolved
@ -0,0 +54,4 @@
auto-load
url="Workers/search"
/>
<fetch-data
Member

FechData

FechData
Contributor

Nombre actualizado

Commit: 3569afdb85

Nombre actualizado Commit: https://gitea.verdnatura.es/hyervoni/salix-front-mindshore/commit/3569afdb85f612696e4c36404402ae92fc460d68
jsegarra marked this conversation as resolved
@ -0,0 +60,4 @@
auto-load
url="Countries"
/>
<fetch-data
Member

FechData

FechData
Contributor

Nombre actualizado

Commit: 3569afdb85

Nombre actualizado Commit: https://gitea.verdnatura.es/hyervoni/salix-front-mindshore/commit/3569afdb85f612696e4c36404402ae92fc460d68
jsegarra marked this conversation as resolved
@ -0,0 +75,4 @@
>
<template #form="{ data }">
<VnRow class="row q-gutter-md q-mb-md">
<div class="col">
Member

Quitar <div class="col">

Quitar `<div class="col">`
Contributor

<div class="col"> removidos

Commit: 71b2f7c8dc

`<div class="col">` removidos Commit: https://gitea.verdnatura.es/hyervoni/salix-front-mindshore/commit/71b2f7c8dccccdaa017044f2c9cdcac4b199f656
jsegarra marked this conversation as resolved
@ -0,0 +78,4 @@
<div class="col">
<VnInput :label="t('Name')" clearable v-model="data.firstName" />
</div>
<div class="col">
Member

Quitar <div class="col">

Quitar `<div class="col">`
Contributor

<div class="col"> removidos

Commit: 71b2f7c8dc

`<div class="col">` removidos Commit: https://gitea.verdnatura.es/hyervoni/salix-front-mindshore/commit/71b2f7c8dccccdaa017044f2c9cdcac4b199f656
jsegarra marked this conversation as resolved
@ -0,0 +84,4 @@
</VnRow>
<VnRow class="row q-gutter-md q-mb-md">
<div class="col">
Member

Ya no hace falta usar <div class="col"> en los VnRow (por lo menos si teneis vuesta rama /dev actualizada)

Ya no hace falta usar `<div class="col">` en los `VnRow` (por lo menos si teneis vuesta rama /dev actualizada)
Contributor

<div class="col"> removidos

Commit: 71b2f7c8dc

`<div class="col">` removidos Commit: https://gitea.verdnatura.es/hyervoni/salix-front-mindshore/commit/71b2f7c8dccccdaa017044f2c9cdcac4b199f656
jsegarra marked this conversation as resolved
@ -0,0 +91,4 @@
clearable
/>
</div>
<div class="col">
Member

Quitar <div class="col">

Quitar `<div class="col">`
Contributor

<div class="col"> removidos

Commit: 71b2f7c8dc

`<div class="col">` removidos Commit: https://gitea.verdnatura.es/hyervoni/salix-front-mindshore/commit/71b2f7c8dccccdaa017044f2c9cdcac4b199f656
jsegarra marked this conversation as resolved
@ -0,0 +101,4 @@
</VnRow>
<VnRow class="row q-gutter-md q-mb-md">
<div class="col">
Member

Quitar <div class="col">

Quitar `<div class="col">`
Contributor

<div class="col"> removidos

Commit: 71b2f7c8dc

`<div class="col">` removidos Commit: https://gitea.verdnatura.es/hyervoni/salix-front-mindshore/commit/71b2f7c8dccccdaa017044f2c9cdcac4b199f656
jsegarra marked this conversation as resolved
@ -0,0 +123,4 @@
</template>
</VnSelectFilter>
</div>
<div class="col">
Member

Quitar <div class="col">

Quitar `<div class="col">`
Contributor

<div class="col"> removidos

Commit: 71b2f7c8dc

`<div class="col">` removidos Commit: https://gitea.verdnatura.es/hyervoni/salix-front-mindshore/commit/71b2f7c8dccccdaa017044f2c9cdcac4b199f656
jsegarra marked this conversation as resolved
@ -0,0 +176,4 @@
</template>
<i18n>
es:
Member

Si alguna de estas traducciones esta de manera global, usarla

Si alguna de estas traducciones esta de manera global, usarla
Contributor

No encontré ninguna de las traducciones de WorkerBasicData dentro del objeto de traducciones globals

No encontré ninguna de las traducciones de `WorkerBasicData` dentro del objeto de traducciones `globals`
worker.patch Outdated
@ -0,0 +1,105 @@
diff --git a/src/components/FormModel.vue b/src/components/FormModel.vue
Member

Eliminar este archivo...

Eliminar este archivo...
Contributor

Eliminado

Commit: 3055db8ef8

Eliminado Commit: https://gitea.verdnatura.es/hyervoni/salix-front-mindshore/commit/3055db8ef8d33ea10e29a985d5b6f4416617911f
jsegarra marked this conversation as resolved
jsegarra requested review from alexm 2024-04-03 12:23:05 +00:00
jsegarra approved these changes 2024-04-04 12:11:04 +00:00
Dismissed
alexm requested changes 2024-04-05 06:22:12 +00:00
Dismissed
alexm left a comment
Member

No hay q subir el package-lock.json ahora usamos pnpm y sigue apareciendo el archivo worker.patch.
Todo lo demas OK

No hay q subir el package-lock.json ahora usamos `pnpm` y sigue apareciendo el archivo `worker.patch`. Todo lo demas OK
jsegarra closed this pull request 2024-04-05 08:24:55 +00:00
jsegarra reopened this pull request 2024-04-05 08:24:58 +00:00
Member

No hay q subir el package-lock.json ahora usamos pnpm y sigue apareciendo el archivo worker.patch.
Todo lo demas OK

He cerrado y vuelto a abrir la PR así eliminamos del listado de archivos modificados el fichero patch

Pues @wbuezas quedaria quitar el fichero que comenta y actualizar con vn:dev

> No hay q subir el package-lock.json ahora usamos `pnpm` y sigue apareciendo el archivo `worker.patch`. > Todo lo demas OK He cerrado y vuelto a abrir la PR así eliminamos del listado de archivos modificados el fichero patch Pues @wbuezas quedaria quitar el fichero que comenta y actualizar con vn:dev
wbuezas added 1 commit 2024-04-05 11:25:09 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
03be831ee5
remove package-lock.json
wbuezas dismissed jsegarra’s review 2024-04-05 11:25:09 +00:00
Reason:

New commits pushed, approval review dismissed automatically according to repository settings

wbuezas added 2 commits 2024-04-05 11:42:22 +00:00
Contributor

No hay q subir el package-lock.json ahora usamos pnpm y sigue apareciendo el archivo worker.patch.
Todo lo demas OK

He cerrado y vuelto a abrir la PR así eliminamos del listado de archivos modificados el fichero patch

Pues @wbuezas quedaria quitar el fichero que comenta y actualizar con vn:dev

Todo listo!

> > No hay q subir el package-lock.json ahora usamos `pnpm` y sigue apareciendo el archivo `worker.patch`. > > Todo lo demas OK > > He cerrado y vuelto a abrir la PR así eliminamos del listado de archivos modificados el fichero patch > > Pues @wbuezas quedaria quitar el fichero que comenta y actualizar con vn:dev Todo listo!
jsegarra requested review from alexm 2024-04-05 11:45:26 +00:00
alexm approved these changes 2024-04-08 06:17:58 +00:00
jsegarra merged commit dafc9f83b9 into dev 2024-04-08 06:19:29 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
5 Participants
Notifications
Total Time Spent: 7 seconds
jsegarra
7 seconds
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#245
No description provided.