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

#7323 workerList #719

Merged
jorgep merged 23 commits from 7323-fineTunningWorker into dev 2024-10-07 06:41:37 +00:00
Conversation 0 Commits 23 Files Changed 16 +11 -6
1 changed files with 11 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit f809e0fb31 - Show all commits

17
src/pages/Worker/Card/WorkerTimeControl.vue
View File

@ -326,16 +326,21 @@ const updateData = async () => {
};
const getMailStates = async (date) => {
const url = `WorkerTimeControls/${route.params.id}/getMailStates`;
const month = date.getMonth() + 1;
const prevMonth = month == 1 ? 12 : month - 1;
let prevMonthStates = [];
jorgep marked this conversation as resolved
jsegarra commented 2024-09-26 11:54:37 +00:00
Review
Copy Link

porque defines aquí la variable en vez de const en la propia llamada?

porque defines aquí la variable en vez de const en la propia llamada?
const params = {
month: date.getMonth() + 1,
month,
year: date.getFullYear(),
};
const { data } = await axios.get(
`WorkerTimeControls/${route.params.id}/getMailStates`,
{ params }
);
workerTimeControlMails.value = data;
const curMonthStates = (await axios.get(url, { params })).data;
jsegarra commented 2024-09-26 11:56:11 +00:00
Outdated
Review
Copy Link

Mmm...no se puede simplificar en una funcion ya que lo que cambia es el valor de params

Mmm...no se puede simplificar en una funcion ya que lo que cambia es el valor de params
jorgep commented 2024-09-26 12:44:52 +00:00
Outdated
Review
Copy Link

Tendría que cambiar el back.

Tendría que cambiar el back.
jsegarra commented 2024-09-26 12:50:17 +00:00
Outdated
Review
Copy Link

Lo que digo que no repetir tanto la llamada a axios si ambas tienen el mismo formato

Lo que digo que no repetir tanto la llamada a axios si ambas tienen el mismo formato
jorgep commented 2024-09-26 12:56:30 +00:00
Outdated
Review
Copy Link

es que para 1 línea no veo la necesidad de crear una fn...

es que para 1 línea no veo la necesidad de crear una fn...
jorgep commented 2024-09-26 13:13:34 +00:00
Outdated
Review
Copy Link

Si lo prefieres así:

const [curMonthStates, prevMonthStates] = await Promise.all([
        axios.get(url, { params }),
        axios.get(url, { params: { ...params, month: prevMonth } }),
    ]);

    workerTimeControlMails.value = curMonthStates.data.concat(prevMonthStates.data);
Si lo prefieres así: ``` const [curMonthStates, prevMonthStates] = await Promise.all([ axios.get(url, { params }), axios.get(url, { params: { ...params, month: prevMonth } }), ]); workerTimeControlMails.value = curMonthStates.data.concat(prevMonthStates.data); ```
jsegarra commented 2024-09-26 16:25:06 +00:00
Outdated
Review
Copy Link

Esto es lo que yo pensaba, pero bueno, lo dejamos así

Esto es lo que yo pensaba, pero bueno, lo dejamos así
prevMonthStates = (await axios.get(url, { params: { ...params, month: prevMonth } }))
jorgep commented 2024-09-23 11:00:39 +00:00
Outdated
Review
Copy Link

Suele aparecer la semana del mes anterior, para poder pintarla del color correcto, hace falta hacer 2 llamadas.

Suele aparecer la semana del mes anterior, para poder pintarla del color correcto, hace falta hacer 2 llamadas.
.data;
workerTimeControlMails.value = curMonthStates.concat(prevMonthStates);
};
const showWorkerTimeForm = (propValue, formType) => {