-
Notifications
You must be signed in to change notification settings - Fork 896
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
UI Split #13303
UI Split #13303
Conversation
This change is to be consistent with the plugins / engines we use (providers, etc.).
We cannot extract locale path from a gettext domains, that is being used in a chain / merge repo. We need to do this ourselves.
This task collects all .po files from all registered ManageIQ plugins / engines and converts them to .json files for them to be consumed by javascript. This is very similar to gettext:po_to_json, except this task takes into account all the engines (plugins), not just the core application.
Depends on companion PR in UI repo.
@@ -1,6 +1,7 @@ | |||
module EvmTestHelper | |||
VMDB_EXCLUDED_SPEC_DIRECTORIES = ["migrations"].freeze | |||
VMDB_EXCLUDED_SPEC_DIRECTORIES = %w(replication migrations).freeze |
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.
@martinpovolny This line is going to have to go back to just being migrations. We previously removed the replication suite.
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.
Nevermind I was reading it backwards I guess :)
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.
Oh no...I was right :P
<github_pr_commenter_batch />Some comments on commits martinpovolny/manageiq@1a9fd56~...db74982 locale/en/manageiq.po
locale/es/manageiq.po
locale/fr/manageiq.po
locale/ja/manageiq.po
locale/manageiq.pot
locale/zh_CN/manageiq.po
|
Checked commits martinpovolny/manageiq@1a9fd56~...db74982 with ruby 2.2.5, rubocop 0.37.2, and haml-lint 0.16.1 Gemfile
config/dictionary_strings.rb
config/yaml_strings.rb
|
Splitting the UI into new repo: manageiq-ui-classic
Companion branch: https://github.com/martinpovolny/manageiq-ui-classic/tree/final_cherryCompanion branch: https://github.com/martinpovolny/manageiq-ui-classic/tree/final_cherry_specsNew repo located at https://github.com/ManageIQ/manageiq-ui-classic