-
Notifications
You must be signed in to change notification settings - Fork 24
Support installing translations for all installed plugins/themes #64
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
Conversation
|
@schlessera provided some feedback on Slack for this: https://wordpress.slack.com/archives/C02RP4T41/p1539759949000100 tldr; Consider using a table for the output which should make it easier to parse the result of each installation. |
|
Current output is inspired by wp language plugin install --all de_DE de_CHwp language plugin install --all de_DE --format=summarywp language plugin install --all de_DE --format=jsonLet me know what you think. If it's good I'm going to do the same for themes and add some tests. |
|
(The |
|
@ocean90 What do you think about combining these two in the "live status messages" (not the table): This would then turn into this instead: This already implies that it will not be installed. Would this make it clearer, or does it worsen overall consistency? |
|
Otherwise, I like the current version and think this is good to go. |
The output next to "Warning" is just one of the |
Co-Authored-By: ocean90 <[email protected]>
Support installing translations for all installed plugins/themes
See #59.