Skip to content

grammar fix pt-BR.json#348

Merged
exelix11 merged 3 commits into
exelix11:masterfrom
Joaosilva27:patch-4
May 5, 2025
Merged

grammar fix pt-BR.json#348
exelix11 merged 3 commits into
exelix11:masterfrom
Joaosilva27:patch-4

Conversation

@Joaosilva27
Copy link
Copy Markdown
Contributor

Sorry for the delay in fixing this, I only recently noticed these small grammar mistakes. Grammar fix for brazilianPortuguese.json has already been done in #345

  • changed "estã instalado" to "está instalado" ("estã" does not exist, "está" is the correct form.)

  • changed "reproduzí-los" to "reproduzi-los" (incorrect accent).

  • changed "Isto" to "Isso" ("Isso" is most commonly used in brazilian portuguese.)

* changed "estã instalado" to "está instalado" ("estã" does not exist, "está" is the correct form.)

* changed "reproduzí-los" to "reproduzi-los" (incorrect accent).

* changed "Isto" to "Isso" ("Isso" is most commonly used in brazilian portuguese.)
@exelix11
Copy link
Copy Markdown
Owner

exelix11 commented May 4, 2025

FYI i also happened to add new strings to the sysmodule yesterday for an upcoming feature, you can see the diff here 23f4d2c#diff-69fc4e7b22303de3d8302f3049e3611a4dfa89dee170a784b54c2f279a1448b1

If you want to update your translation too you can do so in this pr

* portuguese and brazilian portuguese translation for the new sysmodule strings used for an upcoming feature
@Joaosilva27
Copy link
Copy Markdown
Contributor Author

Done. Added translation support to both portuguese and brazilian portuguese.

I can also do it for european spanish if you wish. If so, should I do it in this PR or create a new one?

@exelix11
Copy link
Copy Markdown
Owner

exelix11 commented May 4, 2025

Here is fine by me

* for european spanish

* also fixed one typo from portuguese.json
@Joaosilva27
Copy link
Copy Markdown
Contributor Author

Done!

@exelix11
Copy link
Copy Markdown
Owner

exelix11 commented May 5, 2025

Looks good. Thank you.

@exelix11 exelix11 merged commit c557be5 into exelix11:master May 5, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants