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

[FIX] Missing app permissions translation #21066

Merged
merged 3 commits into from
Mar 22, 2021

Conversation

d-gubert
Copy link
Member

Proposed changes (including videos or screenshots)

Add missing translations for some app permissions

Issue(s)

Steps to test or reproduce

Further comments

@@ -464,6 +464,7 @@
"Apps_Marketplace_Uninstall_Subscribed_App_Prompt": "This app has an active subscription and uninstalling will not cancel it. If you'd like to do that, please modify your subscription before uninstalling.",
"Apps_Permissions_Review_Modal_Title": "This app requires the following permissions",
"Apps_Permissions_No_Permissions_Required": "The App does not require additional permissions",
"Apps_Permissions_cloud_workspace-token": "Interact with Cloud Services on behalf of this server",
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@geekgonecrazy not sure about the wording here... are we calling this the "Cloud Services"? 🤔

Copy link
Contributor

@geekgonecrazy geekgonecrazy Mar 16, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is accurate. Not sure what alternative would be. Since it does actually mean could talk to any of our cloud services.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just wondering if people will know what we're referring to 😛

@d-gubert d-gubert added this to the 3.13.0 milestone Mar 19, 2021
@d-gubert d-gubert marked this pull request as ready for review March 19, 2021 19:37
@ggazzo ggazzo merged commit 6acb8ed into develop Mar 22, 2021
@ggazzo ggazzo deleted the apps/fix-permission-translations branch March 22, 2021 21:19
@sampaiodiego sampaiodiego mentioned this pull request Apr 3, 2021
13 tasks
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