-
Notifications
You must be signed in to change notification settings - Fork 108
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
657 automate localizations #4003
Conversation
99933bf
to
443b854
Compare
283123e
to
c2f41c7
Compare
a95a5fe
to
f138ed4
Compare
apps/dashboard/Gemfile
Outdated
@@ -39,6 +39,10 @@ end | |||
group :development do | |||
# Access an IRB console on exception pages or by using <%= console %> in views | |||
# gem 'web-console', '~> 2.0' | |||
gem 'i18n-tasks' | |||
gem 'highline', '2.1.0' | |||
gem 'easy_translate' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'easy_translate' and 'ruby-openai' bring in gem incompatibilities with Ruby 2.7. However, running bundle exec i18n-tasks translate-missing
(which requires these two gems) translates zero keys. Do we need this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need this?
I think longer term we will. Maybe we just can't automatic updates this until we drop 2.7 support?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good to me. Though note that #657 was about automating it as well so maybe that wasn't quite a good first issue and is indeed 2 or 3 tasks.
Fixes #657