5241-virtual_pos #44

Merged
alexandre merged 6 commits from 5241-virtual_pos into dev 2023-03-23 13:53:46 +00:00
Contributor

Refs #5241

Refs #5241
joan added 4 commits 2023-03-21 10:17:57 +00:00
gitea/salix-front/pipeline/head This commit looks good Details
526a7dd628
feature(POS): added responsive table
gitea/salix-front/pipeline/head This commit looks good Details
19a0e63969
Added unit test
gitea/salix-front/pipeline/head This commit looks good Details
26a7e58c05
Added missing translation
joan added 1 commit 2023-03-21 10:18:06 +00:00
gitea/salix-front/pipeline/head This commit looks good Details
ba9b46e10d
Merge branch 'dev' into 5241-virtual_pos
joan requested review from alexandre 2023-03-21 10:19:49 +00:00
joan added the
CR / Tests passed
label 2023-03-21 10:36:57 +00:00
alexandre requested changes 2023-03-23 09:39:51 +00:00
@ -195,3 +198,3 @@
<i18n>
es:
You didn't enter any filter: No has introducido ningún filtro
No filters applied: No se han aplicado filtros
Contributor

No seria millor posar les traduccions en els fitxers src/i18n/en/index.js i src/i18n/es/index.js?

No seria millor posar les traduccions en els fitxers *src/i18n/en/index.js* i *src/i18n/es/index.js*?
alexandre marked this conversation as resolved
alexandre requested changes 2023-03-23 13:00:28 +00:00
alexandre left a comment
Contributor

jo en general ho vec bé, no sé si al ser una secció nova caldria també un E2E

jo en general ho vec bé, no sé si al ser una secció nova caldria també un E2E
CHANGELOG.md Outdated
@ -13,0 +12,4 @@
- (Clientes) => Añadida nueva sección "Pagos Web" para gestionar los pagos de todos los clientes
- (Tickets) => Añadida opción en el menú desplegable del ticket para enviar SMS al cliente
- (Reclamaciones) => Añadida nueva sección "Registros de auditoría"
- (Trabajadores) => Añádido módulo de trabajadores
Contributor

Llevar accent en añadido

Llevar accent en añadido
@ -0,0 +29,4 @@
.onOk((row) => (row.isConfirmed = true));
}
async function confirmTransaction({ id }) {
Contributor

és correcte passar-li com a parametre el id dins de claus?

és correcte passar-li com a parametre el id dins de claus?
Author
Contributor

Si, el parámetre que li arriva es un objecte data, i ahí el que estic fent es desestructurar el objecte per a extraure directament les propietats

Igual com fer

const { id } = data

Si, el parámetre que li arriva es un objecte data, i ahí el que estic fent es desestructurar el objecte per a extraure directament les propietats Igual com fer `const { id } = data`
joan added 1 commit 2023-03-23 13:36:37 +00:00
gitea/salix-front/pipeline/head This commit looks good Details
61fde3d435
Updated changelog
joan requested review from alexandre 2023-03-23 13:40:12 +00:00
alexandre approved these changes 2023-03-23 13:53:35 +00:00
alexandre merged commit 7cada9ba36 into dev 2023-03-23 13:53:46 +00:00
alexandre deleted branch 5241-virtual_pos 2023-03-23 13:53:46 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 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#44
No description provided.