feat: #8443 created vehicle events #1379

Open
provira wants to merge 20 commits from 8443-vehicleEvent into dev
Member
No description provided.
provira added 1 commit 2025-02-12 11:28:07 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
a261de39b4
feat: refs #8443 created vehicle events
provira changed title from feat: #8443 created vehicle events to WIP: feat: #8443 created vehicle events 2025-02-12 11:28:13 +00:00
provira added 1 commit 2025-02-12 11:29:37 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
80b0efac5a
Merge branch 'dev' into 8443-vehicleEvent
provira added 2 commits 2025-02-12 11:33:02 +00:00
provira added 1 commit 2025-02-13 06:49:47 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
398f76c6e7
feat: refs #8443 added select in form for vehicle state
provira added 1 commit 2025-02-21 06:49:52 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
ff24971df9
Merge branch 'dev' into 8443-vehicleEvent
provira added 1 commit 2025-02-24 12:03:25 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
33c268ddb9
Merge branch 'dev' into 8443-vehicleEvent
provira added 1 commit 2025-03-03 09:46:00 +00:00
gitea/salix-front/pipeline/pr-dev This commit is unstable Details
ecdd1a95df
Merge branch 'dev' into 8443-vehicleEvent
provira added 1 commit 2025-03-03 12:09:09 +00:00
gitea/salix-front/pipeline/pr-dev This commit is unstable Details
8ffc3e8f2d
feat: refs #8443 added notifications and translations
provira changed title from WIP: feat: #8443 created vehicle events to feat: #8443 created vehicle events 2025-03-07 09:50:11 +00:00
provira requested review from jgallego 2025-03-07 09:50:15 +00:00
jgallego requested changes 2025-03-07 11:32:00 +00:00
@ -0,0 +92,4 @@
show-work-weeks
no-outside-days
no-active-date
:weekdays="[1, 2, 3, 4, 5, 6, 0]"
Owner

buscar libreria o forma de no poner numeros, si no lo hemos usado ya, esto sera utilizado por mas secciones

buscar libreria o forma de no poner numeros, si no lo hemos usado ya, esto sera utilizado por mas secciones
provira marked this conversation as resolved
@ -0,0 +198,4 @@
align-items: center;
background-color: $primary;
font-weight: bold;
font-size: 16px;
Owner

no podem anar posant px en seccions concretes, busca forma de tindreu generic

no podem anar posant px en seccions concretes, busca forma de tindreu generic
provira marked this conversation as resolved
@ -0,0 +9,4 @@
cy.get('.q-page-sticky > div > .q-btn').click();
cy.dataCy('Started_inputDate').type('05/02/2001');
cy.dataCy('Finished_inputDate').type('08/02/2001');
cy.get(':nth-child(5) > [label="Description"] > .q-field > .q-field__inner > .q-field__control > .q-field__control-container > [data-cy="Description_input"]').type('Test');
Owner

selector muy largo usar cy sin depender del resto de la ruta

selector muy largo usar cy sin depender del resto de la ruta
provira marked this conversation as resolved
@ -0,0 +23,4 @@
cy.get('.q-mt-lg > .q-btn--standard').click();
});
it('should delete a vehicle event', () => {
Owner

los it dependen unos de otros, haz uno, que cree, consulte, y borre.
Así evitas tb el tener que hacer login cada vez.
@alexm ho veus ok?

los it dependen unos de otros, haz uno, que cree, consulte, y borre. Así evitas tb el tener que hacer login cada vez. @alexm ho veus ok?
Member

Sii millor junt, aixina no depenen uns dels altres

Sii millor junt, aixina no depenen uns dels altres
provira marked this conversation as resolved
provira added 1 commit 2025-03-21 11:33:02 +00:00
provira added 1 commit 2025-03-21 11:33:07 +00:00
gitea/salix-front/pipeline/pr-dev This commit is unstable Details
018084c7c4
Merge branch 'dev' into 8443-vehicleEvent
provira requested review from jgallego 2025-03-21 11:54:37 +00:00
jgallego requested review from jsegarra 2025-03-21 12:16:24 +00:00
jsegarra requested changes 2025-03-21 15:20:22 +00:00
Dismissed
@ -0,0 +63,4 @@
vehicleFormData.value.vehicleFk = route.params.id;
vehicleFormData.value.userFk = user.value.id;
if (isNew.value) {
Member

revisa esta condicion porque hay codigo repeidto que se puede evitar

revisa esta condicion porque hay codigo repeidto que se puede evitar
provira marked this conversation as resolved
@ -0,0 +81,4 @@
const deleteVehicleEvent = async () => {
if (!props.event) return;
await axios.delete(`VehicleEvents/${props.event?.id}`);
notify(t('globals.dataSaved'), 'positive');
Member

Estas 3 líneas son iguales a la de arriba, una función que las incluya?

Estas 3 líneas son iguales a la de arriba, una función que las incluya?
provira marked this conversation as resolved
@ -0,0 +114,4 @@
emit('refresh-events');
};
onMounted(() => {
Member

onMounted no lo dejamos al final
https://vuejs.org/style-guide/rules-recommended

onMounted no lo dejamos al final https://vuejs.org/style-guide/rules-recommended
provira marked this conversation as resolved
@ -0,0 +142,4 @@
url="VehicleStates"
v-model="vehicleFormData.vehicleStateFk"
:label="t('globals.state')"
option-value="id"
Member

esta linea no hace falta

esta linea no hace falta
provira marked this conversation as resolved
@ -0,0 +29,4 @@
const openForm = (data, isBtnAdd) => {
const { date = null, isNewMode, event } = data;
vehicleEventsFormProps.date = date;
Member

No me acaba de convencer pero que te parece la solucion

({ date = null, isNewMode, event } = data);
Object.assign(vehicleEventsFormProps, { date, isNewMode, event });

No me acaba de convencer pero que te parece la solucion ``` ({ date = null, isNewMode, event } = data); Object.assign(vehicleEventsFormProps, { date, isNewMode, event }); ```
Author
Member

Me gusta más así que antes, funciona igual y queda mas limpio

Me gusta más así que antes, funciona igual y queda mas limpio
jsegarra marked this conversation as resolved
@ -0,0 +125,4 @@
</script>
<template>
<QForm @submit="onSubmit()">
Member

Porque no FormModel?

Porque no FormModel?
jsegarra marked this conversation as resolved
@ -0,0 +1,147 @@
<script setup>
Member

Uff...esto está duplicado de ZoneCalendar, al 80%

Uff...esto está duplicado de ZoneCalendar, al 80%
Member

Habría que proponer de extraer la funcionalidad común y/o un componente común.
Ahora son 2 casos, pero en el futuro quien sabe
@jgallego Lo dejamos así y crear tarea para ZoneCalendar y VehicleCalendar o abordamos en esta y creamos tarea "Refactor" para ZoneCalendar?

Habría que proponer de extraer la funcionalidad común y/o un componente común. Ahora son 2 casos, pero en el futuro quien sabe @jgallego Lo dejamos así y crear tarea para ZoneCalendar y VehicleCalendar o abordamos en esta y creamos tarea "Refactor" para ZoneCalendar?
Owner

Si esta repetido, directamente hagamos bien la seccion

Si esta repetido, directamente hagamos bien la seccion
Member

He creado la tarea a posterior de esta https://redmine.verdnatura.es/issues/8797

He creado la tarea a posterior de esta https://redmine.verdnatura.es/issues/8797
Member

Si esta repetido, directamente hagamos bien la sección

@provira Te propongo lo siguiente: dejar los test , y una vez conseguidos, hacer el cambio en Vehicle*

> Si esta repetido, directamente hagamos bien la sección @provira Te propongo lo siguiente: dejar los test ✅, y una vez conseguidos, hacer el cambio en Vehicle*
jsegarra marked this conversation as resolved
@ -0,0 +1,196 @@
<script setup>
Member

3/4 de lo mismo que antes

3/4 de lo mismo que antes
jsegarra marked this conversation as resolved
@ -182,7 +184,7 @@ watch(_year, (newValue) => {
no-outside-days
:selected-dates="calendarEventDates"
no-active-date
:weekdays="[1, 2, 3, 4, 5, 6, 0]"
Member

Asi es como estaba en la docu de quasar, pero el cambio lo veo bien

Asi es como estaba en la docu de quasar, pero el cambio lo veo bien
jsegarra marked this conversation as resolved
@ -0,0 +16,4 @@
cy.get('.q-current-day > .q-calendar-month__day--content > .q-btn').click();
cy.dataCy('Started_inputDate').clear().type('03/02/2001');
cy.dataCy('Finished_inputDate').clear().type('15/03/2001');
cy.get(':nth-child(5) > [label="Description"] > .q-field > .q-field__inner > .q-field__control > .q-field__control-container > [data-cy="Description_input"]').clear().type('Test2');
Member

Simplificar con cy.get(':nth-child(5)').find([data-cy]), se puede?

Simplificar con cy.get(':nth-child(5)').find([data-cy]), se puede?
Author
Member

Si, se puede, ya esta cambiado

Si, se puede, ya esta cambiado
jsegarra marked this conversation as resolved
provira added 1 commit 2025-03-26 07:33:49 +00:00
gitea/salix-front/pipeline/pr-dev This commit is unstable Details
47779437d3
feat: refs #8443 added EntityCalendar and EntityCalendarGrid
provira added 1 commit 2025-03-26 07:34:10 +00:00
gitea/salix-front/pipeline/pr-dev This commit is unstable Details
d12cd08e08
Merge branch 'dev' into 8443-vehicleEvent
provira added 1 commit 2025-03-26 08:34:34 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
4e0922e989
Merge branch 'dev' into 8443-vehicleEvent
provira requested review from jsegarra 2025-03-26 09:04:12 +00:00
jsegarra requested changes 2025-03-28 11:04:46 +00:00
Dismissed
@ -0,0 +33,4 @@
const weekdayStore = useWeekdayStore();
const weekDays = useWeekdaysOrder();
const calendarRef = ref(null);
const today = ref(date.formatDate(Date.vnNew(), 'YYYY-MM-DD'));
Member

Si esta linea y la 42 son iguales, quizás extraer una función

Si esta linea y la 42 son iguales, quizás extraer una función
provira marked this conversation as resolved
@ -0,0 +45,4 @@
return `${weekdayStore.getLocaleMonths[props.month - 1].locale} ${props.year}`;
});
const isToday = (timestamp) => {
Member

Aqui no has destructurado pero en la siguiente si, unificar criterio. Propuesta, mejor como getEventByTimestamp

Aqui no has destructurado pero en la siguiente si, unificar criterio. Propuesta, mejor como getEventByTimestamp
provira marked this conversation as resolved
@ -0,0 +27,4 @@
const lastDay = ref(Date.vnNew());
const months = ref([]);
const arrayData = useArrayData(props.dataKey);
const { store } = arrayData;
Member

store no se usa

store no se usa
provira marked this conversation as resolved
@ -0,0 +61,4 @@
onMounted(() => {
if (props.event) {
vehicleFormData.value = { ...props.event };
Member

Esto es lo mismo que props.event, no?

Esto es lo mismo que props.event, no?
Author
Member

Si, es lo mismo. Ahora se usa directamente props.event.

Si, es lo mismo. Ahora se usa directamente props.event.
jsegarra marked this conversation as resolved
@ -0,0 +132,4 @@
<VnInputDate :label="t('Finished')" v-model="vehicleFormData.finished" />
</VnRow>
<VnRow>
<VnInput
Member

Sacar descripcion de Vnrow

Sacar descripcion de Vnrow
provira marked this conversation as resolved
@ -0,0 +3,4 @@
const emit = defineEmits(['onDateSelected']);
const getEventAttrs = ({ day }) => ({
Member

revisamos

revisamos
Member

Quitar

Quitar
provira marked this conversation as resolved
@ -0,0 +1,97 @@
<script setup>
import { ref, watch, onMounted } from 'vue';
Member

onMounted no se usa

onMounted no se usa
provira marked this conversation as resolved
Member

Moviendo descripcion fuera de VnRow image

Moviendo descripcion fuera de VnRow ![image](/attachments/9dd21479-0532-4022-9d7f-cace84b68abc)
provira added 1 commit 2025-03-31 07:26:31 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
70c8780782
fix: refs #8443 code and form view fixes
provira requested review from jsegarra 2025-03-31 07:26:42 +00:00
jsegarra requested changes 2025-03-31 07:44:10 +00:00
Dismissed
@ -0,0 +8,4 @@
import { useWeekdayStore } from 'src/stores/useWeekdayStore';
import useWeekdaysOrder from 'src/composables/getWeekdays';
const formatDate = (dateToFormat, format = 'YYYY-MM-DD') => {
Member

si haces ({...}) queda mas elegante. Los parentesis es sinonimo de hacer return
NOTA: los 3 puntos(...) corresponden al contenido existente

si haces ({...}) queda mas elegante. Los parentesis es sinonimo de hacer return NOTA: los 3 puntos(...) corresponden al contenido existente
provira marked this conversation as resolved
@ -0,0 +9,4 @@
import useWeekdaysOrder from 'src/composables/getWeekdays';
const formatDate = (dateToFormat, format = 'YYYY-MM-DD') => {
return date.formatDate(dateToFormat, format);
Member

He creado redmine para estandarizar esta función/lógica ya que hay 17 resultados en 9 archivos

He creado redmine para estandarizar esta función/lógica ya que hay 17 resultados en 9 archivos
Member
https://redmine.verdnatura.es/issues/8836
jsegarra marked this conversation as resolved
@ -0,0 +38,4 @@
const weekDays = useWeekdaysOrder();
const calendarRef = ref(null);
const today = ref(formatDate(Date.vnNew()));
const todayTimestamp = computed(() => {
Member

Esto también lo incluyo en el redmine porque hay 7 y 7

Esto también lo incluyo en el redmine porque hay 7 y 7
jsegarra marked this conversation as resolved
@ -0,0 +82,4 @@
<template>
<QCalendarMonthWrapper
style="height: 290px; width: 290px"
Member

De los 4 usos que le damos a este componente, solo 1 no lo tiene, curioso
Te parece que definamos una regla CSS global para este componente?

De los 4 usos que le damos a este componente, solo 1 no lo tiene, curioso Te parece que definamos una regla CSS global para este componente?
Member

Se que antes estaba, pero lo podemos dejar mejor

Se que antes estaba, pero lo podemos dejar mejor
Author
Member

El que no tiene estilo es porque es el calendario de semanas del Time Control de Worker. Si se le ponen los mismos estilos no cabe en el menú lateral

El que no tiene estilo es porque es el calendario de semanas del Time Control de Worker. Si se le ponen los mismos estilos no cabe en el menú lateral
jsegarra requested changes 2025-03-31 07:49:12 +00:00
Dismissed
@ -0,0 +140,4 @@
<VnInput
v-model="vehicleFormData.description"
:label="t('globals.description')"
type="text"
Member

No es necesario el text

No es necesario el text
provira marked this conversation as resolved
provira added 1 commit 2025-03-31 11:04:34 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
7d6b3e27b4
refactor: refs #8443 improved code
provira added 1 commit 2025-04-01 09:02:34 +00:00
provira requested review from jsegarra 2025-04-01 09:35:03 +00:00
jsegarra approved these changes 2025-04-01 13:12:45 +00:00
Dismissed
provira added 1 commit 2025-04-04 13:06:09 +00:00
gitea/salix-front/pipeline/pr-dev This commit is unstable Details
0a299b0fe5
feat: refs #8443 used data-cy for select in test
provira dismissed jsegarra’s review 2025-04-04 13:06:10 +00:00
Reason:

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

provira added 1 commit 2025-04-15 12:38:09 +00:00
jsegarra approved these changes 2025-04-16 09:43:11 +00:00
All checks were successful
gitea/salix-front/pipeline/pr-dev This commit looks good
Required
Details
This pull request has changes conflicting with the target branch.
  • src/router/modules/route.js
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#1379
No description provided.