Skip to content
This repository has been archived by the owner on Sep 26, 2024. It is now read-only.

Fix candidata a presidencia feminino #42

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

brendamoreira
Copy link
Contributor

Para candidatas mulheres à presidência aparecia "undefined", conforme imagem abaixo:

Screenshot 2021-10-18 at 15 15 59

Alterei para:

Screenshot 2021-10-18 at 15 29 37

obs: fiquei na dúvida quanto a colocar "Presidente" ou "Presidenta", então acabei por colocar "Presidente" mesmo, mas posso alterar caso seja melhor

Copy link
Member

@ogecece ogecece left a comment

Choose a reason for hiding this comment

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

Muito obrigado @brendamoreira !


English

Thank you very much @brendamoreira !

@@ -137,6 +137,7 @@ module.exports = {
"POLICIAL CIVIL": "POLICIAL CIVIL",
"POLICIAL MILITAR": "POLICIAL MILITAR",
PREFEITO: "PREFEITA",
PRESIDENTE: "PRESIDENTE",
Copy link
Member

Choose a reason for hiding this comment

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

Como presidenta também está correto pela norma culta e prezando pela consistência com a maioria das profissões, sugiro mudar para PRESIDENTA.


English

As the feminine form is also correct by the formal language and regarding consistency with most of the professions I suggest changing it to PRESIDENTA.

@ogecece ogecece added the hacktoberfest-accepted Hacktoberfest label label Oct 29, 2021
@lguima
Copy link

lguima commented Oct 9, 2022

@giuliocc acho que podemos encerrar essa PR, pois a estrutura do projeto mudou, esse arquivo não existe mais.

@trolzen
Copy link

trolzen commented Oct 12, 2022

Indeed, this should be closed.
@giuliocc

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

Successfully merging this pull request may close these issues.

4 participants