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

stats plugin text #15735

Merged
merged 1 commit into from
May 23, 2017
Merged

stats plugin text #15735

merged 1 commit into from
May 23, 2017

Conversation

brianteeman
Copy link
Contributor

@brianteeman brianteeman commented May 2, 2017

Improve the text of the message

specifically it is wrong to say "this plugin" in the context of where the message is displayedd

@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators PR-staging labels May 2, 2017
@@ -24,7 +24,7 @@ PLG_SYSTEM_STATS_MODE_OPTION_ALWAYS_SEND="Always send"
PLG_SYSTEM_STATS_MODE_OPTION_NEVER_SEND="Never send"
PLG_SYSTEM_STATS_MODE_OPTION_ON_DEMAND="On demand"
PLG_SYSTEM_STATS_MSG_ALLOW_SENDING_DATA="Enable Joomla Statistics?"
PLG_SYSTEM_STATS_MSG_JOOMLA_WANTS_TO_SEND_DATA="In order to better understand our install base and end user environments, this plugin has been created to send those statistics back to a Joomla! controlled central server. No identifying data is captured at any point. You can change these settings later from Plugins > System - Joomla! Statistics."
PLG_SYSTEM_STATS_MSG_JOOMLA_WANTS_TO_SEND_DATA="In order to better understand our install base and end user environments it is helpful if you send some site information back to a Joomla! controlled central server. No identifying data is captured at any point. You can change these settings later from Plugins > System - Joomla! Statistics."
Copy link
Contributor

Choose a reason for hiding this comment

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

I would put a comma after environments, otherwise, it looks good.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

there is no need for the comma

@Quy
Copy link
Contributor

Quy commented May 4, 2017

I have tested this item ✅ successfully on f3f7ed4


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/15735.

1 similar comment
@tonypartridge
Copy link
Contributor

I have tested this item ✅ successfully on f3f7ed4


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/15735.

@joomla-cms-bot joomla-cms-bot removed Language Change This is for Translators PR-staging labels May 23, 2017
@ghost
Copy link

ghost commented May 23, 2017

RTC after two successful tests.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label May 23, 2017
@rdeutz rdeutz merged commit 74e81f4 into joomla:staging May 23, 2017
@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators PR-staging and removed RTC This Pull Request is Ready To Commit labels May 23, 2017
@rdeutz rdeutz added this to the Joomla 3.7.3 milestone May 23, 2017
@brianteeman
Copy link
Contributor Author

Thanks

@brianteeman brianteeman deleted the stats branch May 23, 2017 20:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Language Change This is for Translators
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants