feat: refs #8304 added remove option to operator #1195

Merged
jtubau merged 27 commits from 8304-workerChangesAndFixes into dev 2025-02-05 12:25:27 +00:00
Member
PR BACK: https://gitea.verdnatura.es/verdnatura/salix/pulls/3353
jtubau added 1 commit 2025-01-10 12:07:34 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
628ed69bdb
feat: refs #8304 added remove option to operator
jtubau added 1 commit 2025-01-10 13:20:40 +00:00
Author
Member

@jgallego @alexm Esta pendiente de tomar decisión de donde incluir la funcionalidad de añadir nota, si la forma propuesta en el redmine o en el right-panel

@jgallego @alexm Esta pendiente de tomar decisión de donde incluir la funcionalidad de añadir nota, si la forma propuesta en el redmine o en el right-panel
Member

Yo haria que el propio input feuese, para ver, crear y modificar. Ya sea solo arriba del calendario o solo en el panel de la derecha.
Pero lo de que este el input arriba del calendario pero luego la nota te la ponga en el panel lo veo muy raro

Yo haria que el propio input feuese, para ver, crear y modificar. Ya sea solo arriba del calendario o solo en el panel de la derecha. Pero lo de que este el input arriba del calendario pero luego la nota te la ponga en el panel lo veo muy raro
Owner

@jtubau después de la reunion de la semana pasada de salix, vamos a dejar a la derecha filtros, todo lo que sean acciones como en este caso, lo ponemos arriba.

@jtubau después de la reunion de la semana pasada de salix, vamos a dejar a la derecha filtros, todo lo que sean acciones como en este caso, lo ponemos arriba.
jtubau added 1 commit 2025-01-21 11:28:28 +00:00
jtubau added 1 commit 2025-01-21 13:44:20 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
83064c3813
feat: refs #8304 add saveUrl prop to VnNotes and implement confirm update functionality
jtubau added 1 commit 2025-01-22 15:20:53 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
3b4af9c749
Merge branch 'dev' into 8304-workerChangesAndFixes
jtubau added 1 commit 2025-01-22 15:29:12 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
8599974437
feat: refs #8304 add required prop to VnNotes and update related components
jtubau added 1 commit 2025-01-22 15:32:45 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
adc9d1630b
refactor: refs #8304 put year and contract selectors in the same line
jtubau added 1 commit 2025-01-23 12:38:30 +00:00
jtubau requested review from jgallego 2025-01-23 13:31:29 +00:00
jtubau added 1 commit 2025-01-23 14:05:29 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
d45bf43f6e
Merge branch 'dev' into 8304-workerChangesAndFixes
jtubau added 1 commit 2025-01-23 14:33:23 +00:00
jgallego requested review from jsegarra 2025-01-23 16:14:22 +00:00
jgallego approved these changes 2025-01-23 16:15:14 +00:00
Dismissed
jsegarra reviewed 2025-01-23 21:38:19 +00:00
@ -181,3 +184,1 @@
/>
</template>
</RightMenu>
<Teleport to="#right-panel" v-if="stateStore.isHeaderMounted()">
Member

Tengo dudas de si Teleport es mejor opción que RightMenu
Cual es la motivación de cambiarlo? El v-if?
PEgale un vistazo a este commit 58b8b2f7b0

Tengo dudas de si Teleport es mejor opción que RightMenu Cual es la motivación de cambiarlo? El v-if? PEgale un vistazo a este commit https://gitea.verdnatura.es/verdnatura/salix-front/commit/58b8b2f7b03fc7bc2f1ce76d11ee107179d5fc2c
jtubau marked this conversation as resolved
jsegarra requested changes 2025-01-23 22:15:27 +00:00
Dismissed
@ -16,12 +16,17 @@ import VnSelect from 'components/common/VnSelect.vue';
import FetchData from 'components/FetchData.vue';
import VnInput from 'components/common/VnInput.vue';
const emit = defineEmits(['onFetch']);
Member

Tengo dudas de este componente porque le hemos metido la lógica de cuando es input, y tenemos que jugar con 2 condiciones
Mi propuesta es definir un componente tipo VnObservation que se comporte como cuando dices que es just-input=true, VnNoteType para el Vnelect y VnNote que tenga solo la lógica del textarea
Pero creo que son demasiados cambios para esta PR

Conclusión: VnNotes depende de 3 condiciones para renderizar que elemento u otro. Desacoplar la lógica

Tengo dudas de este componente porque le hemos metido la lógica de cuando es input, y tenemos que jugar con 2 condiciones Mi propuesta es definir un componente tipo VnObservation que se comporte como cuando dices que es just-input=true, VnNoteType para el Vnelect y VnNote que tenga solo la lógica del textarea Pero creo que son demasiados cambios para esta PR Conclusión: VnNotes depende de 3 condiciones para renderizar que elemento u otro. Desacoplar la lógica
Member

Yo lo dejaria en 2. Un VnNoteInput y un VnNote.
Donde el VnNoteInput seria el textArea.
Y el VnNote tener la logica, y este dividido en 2 partes.
Arriba un slot que por defecto contenga el VnNoteInput
Y abajo el VnPaginate

Yo lo dejaria en 2. Un VnNoteInput y un VnNote. Donde el VnNoteInput seria el textArea. Y el VnNote tener la logica, y este dividido en 2 partes. Arriba un slot que por defecto contenga el VnNoteInput Y abajo el VnPaginate
Member

Por tanto, lo dejamos así y creamos tarea con los comentarios que hemos puesto
Importante: referenciar la PR y ambos comentarios en la descripción de la tarea. Indicando: que la funcionalidad que fusionemos en esta PR debe quedar en el mismo estado. Añadir estos como checklist

Por tanto, lo dejamos así y creamos tarea con los comentarios que hemos puesto *Importante*: referenciar la PR y ambos comentarios en la descripción de la tarea. *Indicando*: que la funcionalidad que fusionemos en esta PR debe quedar en el mismo estado. Añadir estos como checklist
@ -23,2 +26,4 @@
addNote: { type: Boolean, default: false },
selectType: { type: Boolean, default: false },
justInput: { type: Boolean, default: false },
required: { type: Boolean, default: false },
Member

Porque prop y no un attr?

Porque prop y no un attr?
jtubau marked this conversation as resolved
@ -31,1 +36,4 @@
const vnPaginateRef = ref();
let originalText;
function handleClick(e) {
Member

Diría que esta lógica ya existe o está implementada en qFormMixin

Diría que esta lógica ya existe o está implementada en qFormMixin
jtubau marked this conversation as resolved
@ -44,0 +64,4 @@
message: t('Are you sure remove this note?'),
},
})
.onOk(() => update())
Member

.onOk(update)

.onOk(update)
jtubau marked this conversation as resolved
@ -44,2 +83,3 @@
onBeforeRouteLeave((to, from, next) => {
if (newNote.text)
if ((newNote.text && !$props.justInput) || (newNote.text !== originalText) && $props.justInput )
Member

ufff...en el limite

ufff...en el limite
Author
Member

que seria sacar la evaluación a una const y pasar la const como condición?
const changesOnNote = (newNote.text && !$props.justInput) || (newNote.text !== originalText) && $props.justInput;
if (changesOnNote)

algo asi?

que seria sacar la evaluación a una const y pasar la const como condición? const changesOnNote = (newNote.text && !$props.justInput) || (newNote.text !== originalText) && $props.justInput; if (changesOnNote) algo asi?
jtubau marked this conversation as resolved
@ -65,2 +105,2 @@
<QCard class="q-pa-xs q-mb-lg full-width" v-if="$props.addNote">
<QCardSection horizontal>
<FetchData
v-if="justInput"
Member

Si usamos selectType para vincularlo con un VnSelect, porque si lo relacionamos con VnInput no hacemos inputType?

Si usamos selectType para vincularlo con un VnSelect, porque si lo relacionamos con VnInput no hacemos inputType?
jtubau marked this conversation as resolved
@ -67,0 +106,4 @@
v-if="justInput"
:url="url"
:filter="filter"
@on-fetch="(data) => (newNote.text = data[0]?.notes, originalText = data[0]?.notes, emit('onFetch', data))"
Member

Demasiadas operaciones para mantener en el apartado template
Como max. 2

Demasiadas operaciones para mantener en el apartado template Como max. 2
Author
Member

function fetchData(data) {
newNote.text = data[0]?.notes;
originalText = data[0]?.notes;
emit('onFetch', data);
}

<FetchData
v-if="justInput"
:url="url"
:filter="filter"
@on-fetch="(data) => (fetchData(data))"
auto-load
/>

así mejor no? le pondrías otro nombre?

function fetchData(data) { newNote.text = data[0]?.notes; originalText = data[0]?.notes; emit('onFetch', data); } <FetchData v-if="justInput" :url="url" :filter="filter" @on-fetch="(data) => (fetchData(data))" auto-load /> así mejor no? le pondrías otro nombre?
jtubau marked this conversation as resolved
@ -67,0 +112,4 @@
<QCard
class="q-pa-xs q-mb-lg full-width"
v-if="$props.addNote || $props.justInput"
:style="$props.justInput ? 'padding-right: 18px; margin-bottom: 2px; box-shadow: none;' : ''"
Member

podemos simplificar con :class="{condicion: "clase CSS"}

podemos simplificar con :class="{condicion: "clase CSS"}
jtubau marked this conversation as resolved
@ -1,8 +1,10 @@
<script setup>
Member

Revisar este componente porque no tiene el mismo estilo en dev que en esta rama

Revisar este componente porque no tiene el mismo estilo en dev que en esta rama
jtubau marked this conversation as resolved
@ -26,3 +30,3 @@
});
onBeforeUnmount(() => stateStore.toggleSubToolbar());
const generalChildCount = () => {
Member

no hace falta definir llaves y return

no hace falta definir llaves y return
jtubau marked this conversation as resolved
@ -19,6 +19,7 @@ const trainsData = ref([]);
const machinesData = ref([]);
const route = useRoute();
const routeId = computed(() => route.params.id);
const selected = ref([]);
Member

No entiendo la propiedad selected que papel hace

No entiendo la propiedad selected que papel hace
Author
Member

para pasarle a crudModel que operario es y que pueda eliminarlo con el botón del subtoolbar

para pasarle a crudModel que operario es y que pueda eliminarlo con el botón del subtoolbar
Member

Okey, veo la necesidad, pero la ejecución es compleja.
Se puede hacer con menos lineas

Okey, veo la necesidad, pero la ejecución es compleja. Se puede hacer con menos lineas
jtubau marked this conversation as resolved
@ -42,2 +43,4 @@
crudModelRef.value.reload();
}
watch(
Member

No veo la utilidad de este watch

No veo la utilidad de este watch
@ -44,3 +59,4 @@
</script>
<template>
Member

Revisar esta sección porque los botones de la subtoolbar se ven raros, mas altos o comprimidos

Revisar esta sección porque los botones de la subtoolbar se ven raros, mas altos o comprimidos
jtubau marked this conversation as resolved
Member

Así se ven los botones
image

Así se ven los botones ![image](/attachments/bd795074-8daa-492d-bbe5-b68c22b3c0b3)
5.4 KiB
jsegarra requested changes 2025-01-23 22:22:49 +00:00
Dismissed
@ -25,1 +12,4 @@
let expectedInsertBody;
let expectedUpdateBody;
function generateWrapper({url = '/test', body = { name: 'Tony', lastName: 'Stark' }, text = null, observationType = null, selectType = false, saveUrl = null, justInput = false }) {
Member

Hay que darle una vuelta a este único parámetro de la función, lo veo muy largo para estar ahi

Hay que darle una vuelta a este único parámetro de la función, lo veo muy largo para estar ahi
jtubau marked this conversation as resolved
jtubau added 1 commit 2025-01-24 07:36:31 +00:00
jtubau dismissed jgallego’s review 2025-01-24 07:36:31 +00:00
Reason:

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

jtubau added 1 commit 2025-01-24 08:20:57 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
864e187a4e
Merge branch 'dev' into 8304-workerChangesAndFixes
jsegarra added 3 commits 2025-01-24 10:10:16 +00:00
jsegarra added 1 commit 2025-01-24 10:11:11 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
d4d4bed74b
feat: refs #8304 workerCalendar Teleport
jsegarra added 1 commit 2025-01-24 10:11:56 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
5ec1d7d311
feat: refs #8304 workerCalendar remove extra VnNotes
jtubau added 1 commit 2025-01-24 10:46:04 +00:00
jtubau added 1 commit 2025-01-24 10:56:45 +00:00
jtubau added 1 commit 2025-01-24 13:38:47 +00:00
jtubau added 1 commit 2025-01-31 08:29:24 +00:00
jtubau added 1 commit 2025-01-31 09:11:14 +00:00
jtubau requested review from jsegarra 2025-01-31 09:13:11 +00:00
jsegarra requested changes 2025-01-31 10:03:46 +00:00
Dismissed
@ -28,1 +36,4 @@
const festiveEventsMap = ref({});
const saveUrl = ref();
const body = {
workerFk: route.params.id,
Member

entiendo la buena practica pero al ser 1 sola linea, menor abajo
2 bien, pero 3 ya no

entiendo la buena practica pero al ser 1 sola linea, menor abajo 2 bien, pero 3 ya no
jsegarra marked this conversation as resolved
jsegarra added 1 commit 2025-02-03 08:27:31 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
035a42a895
Merge branch 'dev' into 8304-workerChangesAndFixes
jsegarra approved these changes 2025-02-03 08:55:36 +00:00
Dismissed
jtubau added 1 commit 2025-02-03 13:29:48 +00:00
jtubau dismissed jsegarra’s review 2025-02-03 13:29:48 +00:00
Reason:

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

jtubau added 1 commit 2025-02-03 13:30:07 +00:00
gitea/salix-front/pipeline/pr-dev Build queued... Details
eabe29ed5c
Merge branch 'dev' into 8304-workerChangesAndFixes
jtubau added 1 commit 2025-02-04 12:01:44 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
5c5a6069ce
Merge branch 'dev' into 8304-workerChangesAndFixes
jsegarra approved these changes 2025-02-04 22:36:42 +00:00
jtubau added 1 commit 2025-02-05 12:23:25 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
c6cf2ed144
Merge branch 'dev' into 8304-workerChangesAndFixes
jtubau merged commit 3aa5bdac92 into dev 2025-02-05 12:25:27 +00:00
jtubau deleted branch 8304-workerChangesAndFixes 2025-02-05 12:25:27 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
4 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#1195
No description provided.