WorkerCalendar #270

Merged
alexm merged 37 commits from :WorkerCalendar into dev 2024-04-16 12:26:20 +00:00
Member
https://redmine.verdnatura.es/issues/6551
hyervoni added 14 commits 2024-04-04 15:02:17 +00:00
wbuezas added 1 commit 2024-04-04 15:02:51 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
cb55115cd2
Remove unused component
wbuezas added 2 commits 2024-04-06 17:10:19 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
4b6038d1d2
Last changes
hyervoni changed title from WIP: WorkerCalendar to WorkerCalendar 2024-04-06 17:11:12 +00:00
jsegarra requested review from jgallego 2024-04-08 11:32:13 +00:00
jsegarra requested review from alexm 2024-04-08 11:32:13 +00:00
jsegarra requested review from jsegarra 2024-04-08 11:32:20 +00:00
jsegarra started working 2024-04-09 06:02:21 +00:00
jsegarra stopped working 2024-04-09 07:43:49 +00:00
1 hour 41 minutes
jgallego requested changes 2024-04-09 08:06:07 +00:00
Dismissed
jgallego left a comment
Owner

En cuanto a los estilos emular salix:

  • Color de fondo del calendario

  • Semanas y fines de semana y letras de la semana con otro color de texto y en mayúsculas

  • Tamaño y espaciado de los numeros

  • Borde del calendario

  • los tipos de ausencias con menos separacion vertical

  • en el worker 1110 en salix se ve la foto, en lilium no.

En cuanto a los estilos emular salix: - [x] Color de fondo del calendario - [ ] Semanas y fines de semana y letras de la semana con otro color de texto y en mayúsculas - [x] Tamaño y espaciado de los numeros - [x] Borde del calendario - [x] los tipos de ausencias con menos separacion vertical - [x] en el worker 1110 en salix se ve la foto, en lilium no.
jsegarra requested changes 2024-04-09 09:38:26 +00:00
Dismissed
@ -0,0 +56,4 @@
if (data.holidays) {
data.holidays.forEach((holiday) => {
const holidayDetail = holiday.detail && holiday.detail.name;
Member

Podemos usar chaining/? hoilday?.detail

Podemos usar chaining/? hoilday?.detail
Member

Esa parte del código la había copiado exactamente igual a salix, aunque me parece bien la idea del chaining y lo apliqué.

Commit: b05c28f571

Esa parte del código la había copiado exactamente igual a `salix`, aunque me parece bien la idea del chaining y lo apliqué. Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/b05c28f571283f3b29403561aac58eb2752df66c
jsegarra marked this conversation as resolved
@ -0,0 +172,4 @@
<QScrollArea class="fit text-grey-8">
<WorkerCalendarFilter
ref="workerCalendarFilterRef"
v-model:business-fk="businessFk"
Member

hace falta v-model?

hace falta v-model?
Member

es que abajo no está :(

es que abajo no está :(
Member

Los v-models en las props businessFk, year y absenceType son necesarios, porque utilizan estos emits:

const emit = defineEmits(['update:businessFk', 'update:year', 'update:absenceType']);

De esta manera pasan la prop y handlean el evento de update.

Los `v-models` en las props `businessFk`, `year` y `absenceType` son necesarios, porque utilizan estos emits: `const emit = defineEmits(['update:businessFk', 'update:year', 'update:absenceType']);` De esta manera pasan la prop y handlean el evento de update.
Member

Okey, ahora lo veo
Gracias

Okey, ahora lo veo Gracias
jsegarra marked this conversation as resolved
@ -0,0 +197,4 @@
<QTooltip max-width="250px">
{{
t(
'To start adding absences, click an absence type from the right menu and then on the day you want to add an absence'
Member

Se que es un rollo, pero podemos mover este parrafo a traducciones como valor e inventar una clave que haga referencia, tipo: toStartAddingAbsences??

Se que es un rollo, pero podemos mover este parrafo a traducciones como valor e inventar una clave que haga referencia, tipo: toStartAddingAbsences??
Member

Claro, moví el texto a una clave de i18n.

Commit: e4166c85fa

Claro, moví el texto a una clave de i18n. Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/e4166c85fabd899cd14c167149c9492bc1dc6bd7
jsegarra marked this conversation as resolved
@ -0,0 +114,4 @@
</QCardSection>
</div>
<div
v-if="contractHolidays"
Member

la condicion if está bien?
Porque en el bloque de antes se usaba contractHolidays y ahora yearHolidays

la condicion if está bien? Porque en el bloque de antes se usaba contractHolidays y ahora yearHolidays
Member

Buena observación, cambie la variable contractHolidays por yearHolidays

Commit: 196679b9e9

Buena observación, cambie la variable `contractHolidays` por `yearHolidays` Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/196679b9e987719569eafcf57963510faf0b4121
jsegarra marked this conversation as resolved
@ -0,0 +125,4 @@
</QCardSection>
<QCardSection class="column items-center" horizontal>
<span>
{{ t('Used') }} {{ yearHolidays.holidaysEnjoyed || 0 }} {{ t('of') }}
Member

Bien, pero si podemos tener traducción con parámetros quizás mejor

Bien, pero si podemos tener traducción con parámetros quizás mejor
Member

Traducciones mejoradas.

Commit: 881c6e2f7f

Traducciones mejoradas. Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/881c6e2f7fa214ffa04e461681ad1a4f9823b223
jsegarra marked this conversation as resolved
@ -0,0 +142,4 @@
<VnSelectFilter
:label="t('Year')"
v-model="selectedYear"
:options="yearList || []"
Member

valor por defecto puede ir en la declaracion?

valor por defecto puede ir en la declaracion?
Member

Valores por default movidos.

Commit: 0941d74d52

Valores por default movidos. Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/0941d74d528fa910df2061f2f5782c83621ff8e9
jsegarra marked this conversation as resolved
@ -0,0 +20,4 @@
<QChip
class="chip"
:selected="selected"
:style="selected ? { backgroundColor: color } : null"
Member

En vez de null poner { backgroundColor: 'black' }. Así está en Salix

En vez de null poner { backgroundColor: 'black' }. Así está en Salix
Member

Color de background cambiado, también apliqué color de texto blancó ya si no en light mode no se veía el texto.

Y decidí renombrar el componente para que se entienda mejor que es.

Commit: fb4f35a378

Color de background cambiado, también apliqué color de texto blancó ya si no en `light mode` no se veía el texto. Y decidí renombrar el componente para que se entienda mejor que es. Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/fb4f35a378545a089111eab4c86e313b8982762a
jsegarra marked this conversation as resolved
@ -0,0 +33,4 @@
</template>
<style scoped lang="scss">
.chip {
Member

Mmm...ya hay una clase en app.scss:82 con la misma regla. Considerar de cambiar

Mmm...ya hay una clase en app.scss:82 con la misma regla. Considerar de cambiar
Member

En el commit de arriba eliminé la clase chip, junto a los otros cambios.

Commit: fb4f35a378

En el commit de arriba eliminé la clase `chip`, junto a los otros cambios. Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/fb4f35a378545a089111eab4c86e313b8982762a
jsegarra marked this conversation as resolved
wbuezas added 1 commit 2024-04-09 11:57:54 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
b05c28f571
Add chaining to holiday detail and holiday type
wbuezas added 1 commit 2024-04-09 12:05:24 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
e4166c85fa
Move add absences text to a i18n variable
wbuezas added 1 commit 2024-04-09 12:07:57 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
196679b9e9
Change v-if variable
wbuezas added 1 commit 2024-04-09 12:15:04 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
0941d74d52
Move default value assignment
wbuezas added 1 commit 2024-04-09 12:21:44 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
fb4f35a378
Rename WorkerDateLabel and change colors
wbuezas added 1 commit 2024-04-09 12:54:32 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
881c6e2f7f
Improve translations
jsegarra requested changes 2024-04-09 13:01:04 +00:00
Dismissed
@ -0,0 +7,4 @@
import QCalendarMonthWrapper from 'src/components/ui/QCalendarMonthWrapper.vue';
import { QCalendarMonth } from '@quasar/quasar-ui-qcalendar/src/index.js';
import '@quasar/quasar-ui-qcalendar/src/QCalendarVariables.sass';
import '@quasar/quasar-ui-qcalendar/src/QCalendarMonth.sass';
Member

Mmm...ya se esta usando en app.scss no?

Mmm...ya se esta usando en app.scss no?
Member

Efectivamente, lo removí.

Commit: 3e5a5dfe1c

Efectivamente, lo removí. Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/3e5a5dfe1c4a238a9ad6292b8bc6b54e50d13cfb
jsegarra marked this conversation as resolved
wbuezas added 1 commit 2024-04-09 13:53:15 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
3e5a5dfe1c
remove import
wbuezas added 1 commit 2024-04-09 14:06:18 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
dfcf6e35d5
Change WorkerEventLabels separations
wbuezas added 1 commit 2024-04-09 17:03:25 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
9ea0a62046
Add border and change background color
wbuezas added 1 commit 2024-04-09 17:13:48 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
c573019038
Sizes and spacings
wbuezas added 1 commit 2024-04-09 17:48:58 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
faa8d05313
Change saturday and sunday days color
wbuezas added 1 commit 2024-04-09 18:20:38 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
9bffc9bb25
classes and size improvements
wbuezas added 1 commit 2024-04-09 18:23:13 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
c3835dfd7a
resolve conflicts
wbuezas added 1 commit 2024-04-09 18:28:20 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
fc4a69b2c6
resolve conflicts in worker routes
Member

En cuanto a los estilos emular salix:

  • [*] Color de fondo del calendario

  • [*] Semanas y fines de semana y letras de la semana con otro color de texto

  • [*] Tamaño y espaciado de los numeros

  • [*] Borde del calendario

  • [*] los tipos de ausencias con menos separacion vertical

  • [*] en el worker 1110 en salix se ve la foto, en lilium no.

@jgallego todos los cambios implementados.

Solo una aclaración, en el último punto sobre la imagen del descriptor no cambié nada, solo me hice pull del backend y funcionó.

> En cuanto a los estilos emular salix: > > - [*] Color de fondo del calendario > > - [*] Semanas y fines de semana y letras de la semana con otro color de texto > > - [*] Tamaño y espaciado de los numeros > > - [*] Borde del calendario > > - [*] los tipos de ausencias con menos separacion vertical > > - [*] en el worker 1110 en salix se ve la foto, en lilium no. @jgallego todos los cambios implementados. Solo una aclaración, en el último punto sobre la imagen del descriptor no cambié nada, solo me hice pull del backend y funcionó.
Owner

En cuanto a los estilos emular salix:

  • Color de fondo del calendario

  • Semanas y fines de semana y letras de la semana con otro color de texto

  • Tamaño y espaciado de los numeros

  • Borde del calendario

  • los tipos de ausencias con menos separacion vertical

  • en el worker 1110 en salix se ve la foto, en lilium no.

@wbuezas he marcado los checkbox que estan ya resueltos y se ha quedado genial, simplemente faltan los numeros de las semanas y las letras de los dias de la semana que van tambien en gris
image
image

> En cuanto a los estilos emular salix: > > - [x] Color de fondo del calendario > > - [ ] Semanas y fines de semana y letras de la semana con otro color de texto > > - [x] Tamaño y espaciado de los numeros > > - [x] Borde del calendario > > - [x] los tipos de ausencias con menos separacion vertical > > - [x] en el worker 1110 en salix se ve la foto, en lilium no. @wbuezas he marcado los checkbox que estan ya resueltos y se ha quedado genial, simplemente faltan los numeros de las semanas y las letras de los dias de la semana que van tambien en gris ![image](/attachments/444b621d-bd33-4685-8ea7-0c4f13a708f1) ![image](/attachments/f7762927-0a88-483a-81d4-7e7e404fe806)
wbuezas added 1 commit 2024-04-10 11:32:12 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
e4a8c45053
Change workweeks colod and capitalize
Member

@wbuezas he marcado los checkbox que estan ya resueltos y se ha quedado genial, simplemente faltan los numeros de las semanas y las letras de los dias de la semana que van tambien en gris

@jgallego se me pasó! Ahí subí un nuevo commit con los cambios que mencionas

> @wbuezas he marcado los checkbox que estan ya resueltos y se ha quedado genial, simplemente faltan los numeros de las semanas y las letras de los dias de la semana que van tambien en gris @jgallego se me pasó! Ahí subí un nuevo commit con los cambios que mencionas
jsegarra requested review from jgallego 2024-04-10 12:17:29 +00:00
jsegarra approved these changes 2024-04-10 12:17:40 +00:00
Dismissed
wbuezas added 1 commit 2024-04-10 12:20:24 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
c881fe6577
Resolve conflicts
wbuezas dismissed jsegarra’s review 2024-04-10 12:20:24 +00:00
Reason:

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

jgallego approved these changes 2024-04-12 05:45:37 +00:00
Dismissed
alexm approved these changes 2024-04-12 05:48:48 +00:00
Dismissed
@ -0,0 +72,4 @@
auto-load
/>
<FetchData
v-if="businessFk"
Member

Este if diria q no hace falta

Este if diria q no hace falta
Member

Quitado

Commit: c12fd77218

Quitado Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/c12fd772184b8d7a26391c817d96a72004c3b392
@ -0,0 +76,4 @@
:url="`Workers/${route.params.id}/contracts`"
:filter="{
fields: ['businessFk', 'businessFk', 'started', 'ended'],
where: { businessFk },
Member

Y este where no debe estar. si no en el desplegable solo me muestra mi contrato actual

Y este where no debe estar. si no en el desplegable solo me muestra mi contrato actual
Member

Quitado

Commit: c12fd77218

También me acabo de dar cuenta que estaba repetido el businessFk en fields

Commit: 3d9d444bb4

Quitado Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/c12fd772184b8d7a26391c817d96a72004c3b392 También me acabo de dar cuenta que estaba repetido el `businessFk` en `fields` Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/3d9d444bb425099ed13aeb709c85a7b87bbb3d38
alexm requested changes 2024-04-12 05:49:14 +00:00
Dismissed
alexm left a comment
Member

Cambios de antes

Cambios de antes
wbuezas added 1 commit 2024-04-12 11:10:24 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
c12fd77218
Remove businessFk from FetchData
wbuezas dismissed jgallego’s review 2024-04-12 11:10:24 +00:00
Reason:

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

wbuezas added 1 commit 2024-04-12 11:12:07 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
3d9d444bb4
remove filter field
wbuezas added 1 commit 2024-04-12 11:14:55 +00:00
alexm reviewed 2024-04-16 09:59:01 +00:00
alexm left a comment
Member

Resolver conflictos y fusionar

Resolver conflictos y fusionar
alexm approved these changes 2024-04-16 09:59:17 +00:00
Dismissed
wbuezas added 1 commit 2024-04-16 11:49:45 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
2ab7a45114
resolve conflicts
wbuezas dismissed alexm’s review 2024-04-16 11:49:45 +00:00
Reason:

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

Member

Resolver conflictos y fusionar

@alexm rama actualizada!

> Resolver conflictos y fusionar @alexm rama actualizada!
jsegarra requested review from alexm 2024-04-16 12:04:27 +00:00
jsegarra requested review from jsegarra 2024-04-16 12:04:27 +00:00
jsegarra approved these changes 2024-04-16 12:04:31 +00:00
alexm approved these changes 2024-04-16 12:26:01 +00:00
alexm merged commit 5d6a10050e into dev 2024-04-16 12:26:20 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
5 Participants
Notifications
Total Time Spent: 1 hour 41 minutes
jsegarra
1 hour 41 minutes
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#270
No description provided.