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 +8 -8
7 changed files with 8 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 18ea23adf1 - Show all commits

1
src/i18n/locale/en.yml
View File

@ -303,6 +303,7 @@ globals:
email: Email
SSN: SSN
fi: FI
deleteConfirmTitle: Delete selected elements
errors:
statusUnauthorized: Access denied
statusInternalServerError: An internal server error has ocurred

2
src/i18n/locale/es.yml
View File

@ -307,6 +307,8 @@ globals:
email: Correo
SSN: NSS
fi: NIF
deleteConfirmTitle: Eliminar los elementos seleccionados
errors:
statusUnauthorized: Acceso denegado
statusInternalServerError: Ha ocurrido un error interno del servidor

2
src/pages/Monitor/MonitorOrders.vue
View File

@ -159,7 +159,7 @@ const openTab = (id) =>
color="primary"
@click="
openConfirmationModal(
$t('salesOrdersTable.deleteConfirmTitle'),
$t('globals.deleteConfirmTitle'),
jorgep marked this conversation as resolved Outdated
jsegarra commented 2024-09-26 12:02:14 +00:00
Outdated
Review
Copy Link

es un texto tan tan genérico, que propongo moverlo a global

es un texto tan tan genérico, que propongo moverlo a global
$t('salesOrdersTable.deleteConfirmMessage'),
removeOrders
)

1
src/pages/Monitor/locale/en.yml
View File

@ -15,7 +15,6 @@ salesOrdersTable:
dateMake: Make date
client: Client
salesPerson: Salesperson
deleteConfirmTitle: Delete selected elements
deleteConfirmMessage: All the selected elements will be deleted. Are you sure you want to continue?
agency: Agency
import: Import

1
src/pages/Monitor/locale/es.yml
View File

@ -15,7 +15,6 @@ salesOrdersTable:
dateMake: Fecha de realización
client: Cliente
salesPerson: Comercial
deleteConfirmTitle: Eliminar los elementos seleccionados
deleteConfirmMessage: Todos los elementos seleccionados serán eliminados. ¿Seguro que quieres continuar?
agency: Agencia
import: Importe

2
src/pages/Worker/Card/WorkerBalance.vue
View File

@ -82,7 +82,7 @@ const columns = computed(() => [
:create="{
urlCreate: 'workerIncomes',
title: t('Create balance'),
onDataSaved: () => $refs.tableRef.reload(),
onDataSaved: () => tableRef.reload(),
jorgep marked this conversation as resolved Outdated
jsegarra commented 2024-09-26 12:04:51 +00:00
Outdated
Review
Copy Link

porque usas $refs si ya tienes tableRef como varaiable

porque usas $refs si ya tienes tableRef como varaiable
formInitialData: {
workerFk: entityId,
},

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

@ -329,16 +329,15 @@ 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 = [];
const params = {
month,
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?
year: date.getFullYear(),
};
const curMonthStates = (await axios.get(url, { params })).data;
prevMonthStates = (await axios.get(url, { params: { ...params, month: prevMonth } }))
.data;
const prevMonthStates = (
await axios.get(url, { params: { ...params, month: prevMonth } })
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í
).data;
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.
workerTimeControlMails.value = curMonthStates.concat(prevMonthStates);
};