verdnatura/salix-front
verdnatura
/
salix-front
2
3
Fork 1
Code Issues Pull Requests 91 Releases Wiki Activity

WorkerCalendar #270

Merged
alexm merged 37 commits from :WorkerCalendar into dev 2024-04-16 12:26:20 +00:00
Conversation 4 Commits 37 Files Changed 13 +2 -2
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit b05c28f571 - Show all commits

4
src/pages/Worker/Card/WorkerCalendar.vue
View File

@ -56,8 +56,8 @@ const onFetchAbsences = (data) => {
if (data.holidays) {
data.holidays.forEach((holiday) => {
const holidayDetail = holiday.detail && holiday.detail.name;
const holidayType = holiday.type && holiday.type.name;
const holidayDetail = holiday?.detail?.name;
jsegarra marked this conversation as resolved Outdated
jsegarra commented 2024-04-09 06:10:54 +00:00
Outdated
Review
Copy Link

Podemos usar chaining/? hoilday?.detail

Podemos usar chaining/? hoilday?.detail
wbuezas commented 2024-04-09 11:59:39 +00:00
Outdated
Review
Copy Link

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
const holidayType = holiday?.type?.name;
const holidayName = holidayDetail || holidayType;
addEvent(