Add menu translations #82

Merged
jsegarra merged 10 commits from wbuezas/hedera-web-mindshore:feature/AddMenuTranslations into 4922-vueMigration 2024-09-09 20:19:32 +00:00
Contributor

Se utilizaron las mismas traducciones para el menú, como para el título de la sección

Se utilizaron las mismas traducciones para el menú, como para el título de la sección
wbuezas added 8 commits 2024-09-09 11:59:28 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
bfbe3621d6
Resolve conflicts
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
e94f8c6b23
Add missed code
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
c53658e6e0
Add empty list
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
2bff9304e8
Add title view translation
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
c41d1430f7
Add menu translations
wbuezas requested review from jsegarra 2024-09-09 11:59:44 +00:00
jsegarra requested changes 2024-09-09 18:04:43 +00:00
jsegarra left a comment
Member

Me falta probar la funcionalidad, pero de momento he visto estas cositas

Me falta probar la funcionalidad, pero de momento he visto estas cositas
@ -83,2 +92,4 @@
noData: 'No data',
confirm: 'Confirm',
delete: 'Delete',
reallyDelete: 'Are you sure you want to delete the line?',
Member

confirmDelete

confirmDelete
@ -96,3 +98,3 @@
<QItem v-if="!item.childs" :to="`/${item.path}`">
<QItemSection>
<QItemLabel>{{ item.description }}</QItemLabel>
<QItemLabel>{{ t(item.description) }}</QItemLabel>
Member

$t serviria en vez de usei18n().t?

$t serviria en vez de usei18n().t?
@ -7,0 +71,4 @@
const formattedData = data.map(i => {
const { amount: quantity, ...item } = i;
const formattedItem = {
Member

entiendo que hacemos esta asignación extra durante la fase de desarrollo para visualizar los datos, pero quizas nos la podemos cargar.
Idem para la linea 50

entiendo que hacemos esta asignación extra durante la fase de desarrollo para visualizar los datos, pero quizas nos la podemos cargar. Idem para la linea 50
Member

Esto pertenece a la otra PR

Esto pertenece a la otra PR
jsegarra marked this conversation as resolved
@ -10,1 +94,4 @@
});
onMounted(async () => {
orderId.value = route.params.id || basketOrderId.value;
Member

en Lilium, usamos const entityId, no?

en Lilium, usamos const entityId, no?
Member

Esto pertenece a la otra PR

Esto pertenece a la otra PR
jsegarra marked this conversation as resolved
jsegarra added 2 commits 2024-09-09 20:19:09 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
929623949b
perf: change code position(clean code)
jsegarra merged commit 057bfae517 into 4922-vueMigration 2024-09-09 20:19:32 +00:00
Sign in to join this conversation.
No reviewers
No Label
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/hedera-web#82
No description provided.