Skip to content
This repository has been archived by the owner on Jan 9, 2023. It is now read-only.

fix: Add portuguese(brazilian) translations #2770

Open
wants to merge 101 commits into
base: master
Choose a base branch
from

Conversation

renato04
Copy link

@renato04 renato04 commented Oct 4, 2021

Fixes #2769

Changes proposed in this pull request:
- Add missing pt-br translations.

@vercel
Copy link

vercel bot commented Oct 4, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/hospitalrun/hospitalrun-frontend/2kW5FsjjkZjyUC3FhErusyWotr3w
✅ Preview: https://hospitalrun-frontend-git-fork-renato04-ptbrt-58e4e5-hospitalrun.vercel.app

@CLAassistant
Copy link

CLAassistant commented Oct 4, 2021

CLA assistant check
All committers have signed the CLA.

Copy link

@RobertoMSousa RobertoMSousa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some typos and missing translations

},
},
imaging: {
label: 'imagem',

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
label: 'imagem',
label: 'Imagem',

Can you make it Uppercase?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes!

status: {
reported: 'Criado',
resolved: 'Resolvido',
all: 'All',

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

By no means I'm an expert on PR-BR but several of those tags seems to be in English 🤔

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All was my bad! I'll fix it

view: 'Ver Incidente',
resolve: 'Resolver Incidente',
dateOfIncident: 'Data do Incidente',
department: 'Departmento',

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo

Suggested change
department: 'Departmento',
department: 'Departamento',

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Departament is Departamento in portguese

image

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

exactly, it's missing an a between the t and the m 😉

category: 'Categoria',
categoryItem: 'Categoria do Item',
description: 'Descrição do Incidente',
patient: 'Patiente',

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

English?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Patient was my bad! I'll Fix it

@@ -0,0 +1,6 @@
export default {
networkStatus: {
offline: 'você está trabalhando no modo offiline',

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
offline: 'você está trabalhando no modo offiline',
offline: 'Você está trabalhando no modo offiline',

export default {
networkStatus: {
offline: 'você está trabalhando no modo offiline',
online: 'você está online',

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
online: 'você está online',
online: 'Você está online',

view: 'Ver Incidente',
resolve: 'Resolver Incidente',
dateOfIncident: 'Data do Incidente',
department: 'Departmento',

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

exactly, it's missing an a between the t and the m 😉

@@ -6,8 +6,13 @@ export default {
viewPatient: 'Ver paciente',
newPatient: 'Novo Paciente',
editPatient: 'Mudar paciente',
updatePatient: 'Update Patient',

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

English?

successfullyUpdated: 'Informação atualizada com sucesso para ',
possibleDuplicatePatient: 'Paciente possivelmente duplicado: ',
duplicatePatientWarning:
'Paciente com informação enctrada na base de dados. Tem certeza que deseja criar este paciente?'

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
'Paciente com informação enctrada na base de dados. Tem certeza que deseja criar este paciente?'
'Paciente com informação entrada na base de dados. Tem certeza que deseja criar este paciente?'

typo

patientRequired: 'Paciente é necessario.',
startDateMustBeBeforeEndDate: 'Horário de início deve ser antes do horário final.',
},
reason: 'Razão',
patient: 'Paciente',
deleteConfirmationMessage: 'Tem a certeza de que deseja eliminar este compromisso?',
successfullyCreated: 'Compromisso criado com sucesso',
successfullyDeleted: 'Compromisso deletedo com sucesso.',

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
successfullyDeleted: 'Compromisso deletedo com sucesso.',
successfullyDeleted: 'Compromisso deletado com sucesso.',

descriptionRequired: 'A descrição é obrigatória. ',
conditionRequired: 'A condição é obrigatória. ',
statusRequired: 'O status é obrigatório. ',
intentRequired: 'Intent is required.',

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

English

description: 'Descrição',
status: 'Status',
condition: 'Condição',
intent: 'Intent',

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

English

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This Intent I left in English because I don't know for sure if the literal translation is the right translation.

Copy link

@tsdamas tsdamas Oct 6, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @renato04,

I did some research about Care Plan intent meaning.

I found that Intent means "Describes the intention of how one or more practitioners intend to deliver care for a particular patient, group or community for a period of time, possibly limited to care for a specific condition or set of conditions."

That being said, I believe in Portuguese the right translation would be Cobertura or Tipo de Cobertura, which means the scope of service that the practitioner can provide to that specific patient. Sometimes the patient has business care plans or personal care plans.

I hope that helps.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @tsdamas.
I'll change it

@@ -5,9 +5,14 @@ export default {
viewPatients: 'Exibir pacientes',
viewPatient: 'Ver paciente',
newPatient: 'Novo Paciente',
editPatient: 'Mudar paciente',
editPatient: 'Alterar paciente',
updatePatient: 'Atualziar paciente',

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
updatePatient: 'Atualziar paciente',
updatePatient: 'Atualizar paciente',

@RobertoMSousa
Copy link

Please follow the semantic for the PR
https://sparkbox.com/foundry/semantic_commit_messages

@renato04 renato04 changed the title Added more pt-br translations fix: Add portuguese(brazilian) translations Oct 6, 2021
@RobertoMSousa
Copy link

@renato04 can you please complete this PR?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update Portuguese translation
4 participants