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

Update plugins.md #6716

Merged
merged 3 commits into from Jan 25, 2018
Merged

Update plugins.md #6716

merged 3 commits into from Jan 25, 2018

Conversation

hendrikschneider
Copy link
Contributor

Updated plugin description

@@ -920,7 +920,7 @@ You can find a few useful plugins at the following locations:

#### Other

- [Analytics for Jekyll](https://github.com/hendrikschneider/jekyll-analytics) by Hendrik Schneider: An effortless way to add various trackers like Google Analytics, Piwik, etc. to your site
- [Analytics for Jekyll](https://github.com/hendrikschneider/jekyll-analytics) by Hendrik Schneider: An effortless way to add various trackers like Google Analytics, Piwik, Matomo, MPulse, etc. to your site
Copy link
Member

Choose a reason for hiding this comment

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

  • Piwik is now Matomo.. so it should not be Piwik, Matomo
  • All docs refer to "MPulse" as "mPulse"
- trackers like Google Analytics, Piwik, Matomo, MPulse, etc. to your site
+ trackers like Google Analytics, Matomo (Piwik), mPulse, etc. to your site

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think it is also important to mention Piwik to show backwards compatibility. Would you accept this description?

An effortless way to add various trackers like Google Analytics, 
Matomo (former Piwik), mPulse, etc. to your site.

Copy link
Member

Choose a reason for hiding this comment

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

yes.. I'm fine with adding an adjective.. formerly sounds more apt though..

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Thank you for your contribution!

@@ -920,7 +920,7 @@ You can find a few useful plugins at the following locations:

#### Other

- [Analytics for Jekyll](https://github.com/hendrikschneider/jekyll-analytics) by Hendrik Schneider: An effortless way to add various trackers like Google Analytics, Piwik, etc. to your site
- [Analytics for Jekyll](https://github.com/hendrikschneider/jekyll-analytics) by Hendrik Schneider: An effortless way to add various trackers like Google Analytics, Matomo (former Piwik), mPulse, etc. to your site.
Copy link

Choose a reason for hiding this comment

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

This should say formerly instead of former.

@ghost
Copy link

ghost commented Jan 25, 2018

@jekyllbot: merge +docs

@jekyllbot jekyllbot merged commit 07ec68a into jekyll:master Jan 25, 2018
jekyllbot added a commit that referenced this pull request Jan 25, 2018
@jekyll jekyll locked and limited conversation to collaborators Jul 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants