Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add translate pt br #1353

Closed
wants to merge 5 commits into from

Conversation

gilcierweb
Copy link

Add Brazilian Portuguese translation, creating all new pt-BR.yml translation files and keys on screens that do not have translation.
Captura de tela de 2024-10-23 16-21-09

@gariasf
Copy link
Contributor

gariasf commented Oct 23, 2024

Here @gilcierweb , I'm not sure if this kind of PRs will be merged at the moment.
CleanShot 2024-10-23 at 21 24 13@2x

#1225

@gilcierweb
Copy link
Author

gilcierweb commented Oct 23, 2024

@gariasf Hello, how are you?
No problem, I'm translating locally.
I think the idea of ​​the project is very good.
to test locally I'm doing this in config/application.rb

   # config/application.rb
   # TODO: check best model for translate ex: www.example.com/books?locale=pt-BR
    config.i18n.load_path += Dir[Rails.root.join('config', 'locales', '*.{rb,yml}')]
    config.i18n.default_locale = :'pt-BR'`

@zachgoll
Copy link
Collaborator

@gariasf thanks for the reference! @gilcierweb as outlined in #1225 we are not accepting this contributions yet.

@zachgoll zachgoll closed this Oct 23, 2024
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.

3 participants