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
Marketplace description language reworked #21773
base: 5.x-dev
Are you sure you want to change the base?
Conversation
"LicenseKeyExpiresSoon": "Your license key expires soon. Please contact %1$s.", | ||
"LicenseKeyIsExpired": "Your license key has expired. Please contact %1$s.", | ||
"MultiServerEnvironmentWarning": "Installing this plugin (or new version of it) must be done manually on each server you are using it on. Otherwise it would only change on one server.", | ||
"AutoUpdateDisabledWarning": "Allow auto-installation of new versions by setting %1$s in %2$s to install or get a new version of this plugin directly.", |
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.
(Assuming here "directly" means ~"non-manually".)
plugins/Marketplace/lang/en.json
Outdated
"LicenseExceeded": "License exceeded", | ||
"LicenseExceededDescription": "The licenses for the following plugins are no longer valid as the number of authorized users for the license is exceeded: %1$s. %2$sYou will not be able to download updates for these plugins. To resolve this issue either delete some users or %3$supgrade the subscription now%4$s.", | ||
"PluginLicenseExceededDescription": "You are not allowed to download this plugin. The license for this plugin is no longer valid as the number of authorized users for the license is exceeded. To resolve this issue either delete some users or upgrade your subscription now.", | ||
"LicenseExceededDescription": "%3$sUpgrade your license%4$s or remove users to keep using %1$s. %2$sThen you can also get new versions.", |
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.
I don't understand what the %2$s
is doing here.
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.
Also adds a <br />
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.
Removing the whitespace then. :)
If you don't want this PR to be closed automatically in 28 days then you need to assign the label 'Do not close'. |
This PR was last updated more than one month ago, maybe it's time to close it. Please check if there is anything we still can do or close this PR. ping @matomo-org/core-reviewers |
Description:
More understandable and translatable strings.
Part-way does away with describing "license" as "subscription" so as to only have one term in use.
Review