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

Sort content by popularity using Google Analytics #5812

Merged
merged 1 commit into from Jan 23, 2017

Conversation

Projects
None yet
6 participants
@jhabdas
Contributor

jhabdas commented Jan 23, 2017

Just came across the jekyll-ga generator plugin from the team that built http://www.healthcare.gov and am adding it for posterity.

@jhabdas jhabdas changed the title from Add jekyll-ga plug-in to Sort content by popularity using Google Analytics Jan 23, 2017

@jhabdas jhabdas referenced this pull request Jan 23, 2017

Open

Publish RubyGem #8

@oe

oe approved these changes Jan 23, 2017

@@ -753,6 +753,7 @@ LESS.js files during generation.
- [Jekyll::Paginate::Category](https://github.com/midnightSuyama/jekyll-paginate-category): Pagination Generator for Jekyll Category.
- [AMP-Jekyll by Juuso Mikkonen](https://github.com/juusaw/amp-jekyll): Generate [Accelerated Mobile Pages](https://www.ampproject.org) of Jekyll posts.
- [Jekyll Art Gallery plugin](https://github.com/alexivkin/Jekyll-Art-Gallery-Plugin): An advanced art/photo gallery generation plugin for creating galleries from a set of image folders. Supports image tagging, thumbnails, sorting, image rotation, post-processing (remove EXIF, add watermark), multiple collections and much more.
- [jekyll-ga](https://github.com/developmentseed/jekyll-ga): A Jekyll plugin that downloads Google Analytics data and adds it to posts. Useful for making a site that lists "most popular" content. [Read the introduction](https://developmentseed.org/blog/google-analytics-jekyll-plugin/) post on the developmentSEED blog.

This comment has been minimized.

@DirtyF

DirtyF Jan 23, 2017

Member

GoogLe -> Google / AnaLytics -> Analytics

@DirtyF

DirtyF Jan 23, 2017

Member

GoogLe -> Google / AnaLytics -> Analytics

@oe

This comment has been minimized.

Show comment
Hide comment
@oe

oe Jan 23, 2017

Member

i don't think we should merge this until the code is published as a rubygem

Member

oe commented Jan 23, 2017

i don't think we should merge this until the code is published as a rubygem

@oe oe added the do-not-merge label Jan 23, 2017

@oe

This comment has been minimized.

Show comment
Hide comment
@oe

oe Jan 23, 2017

Member

@jhabdas by the way, can you confirm this plugin works with the latest jekyll version(s)?

Member

oe commented Jan 23, 2017

@jhabdas by the way, can you confirm this plugin works with the latest jekyll version(s)?

@crispgm

This comment has been minimized.

Show comment
Hide comment
@crispgm

crispgm Jan 23, 2017

Member

@pnn as a matter of fact, lots of plugins listed are not published as a gem and even some of them are just gist codes.

Member

crispgm commented Jan 23, 2017

@pnn as a matter of fact, lots of plugins listed are not published as a gem and even some of them are just gist codes.

@oe oe removed the do-not-merge label Jan 23, 2017

@oe

This comment has been minimized.

Show comment
Hide comment
@oe

oe Jan 23, 2017

Member

@crispgm right, forgot about that. the plugin list will (hopefully) be replaced soon anyways

Member

oe commented Jan 23, 2017

@crispgm right, forgot about that. the plugin list will (hopefully) be replaced soon anyways

@DirtyF

This comment has been minimized.

Show comment
Hide comment
@DirtyF

DirtyF Jan 23, 2017

Member

@pnn FYI https://github.com/jekyll/plugins has a type variable to distinguish gems from ruby scripts, but we index all types of plugins.

Member

DirtyF commented Jan 23, 2017

@pnn FYI https://github.com/jekyll/plugins has a type variable to distinguish gems from ruby scripts, but we index all types of plugins.

@parkr

parkr approved these changes Jan 23, 2017

With @DirtyF's comment addressed, :shipit:

@DirtyF

DirtyF approved these changes Jan 23, 2017

@DirtyF

This comment has been minimized.

Show comment
Hide comment
@DirtyF

DirtyF Jan 23, 2017

Member

Thanks @jhabdas.

@jekyllbot: merge +doc

Member

DirtyF commented Jan 23, 2017

Thanks @jhabdas.

@jekyllbot: merge +doc

@jekyllbot jekyllbot merged commit 9792eda into jekyll:master Jan 23, 2017

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@jhabdas jhabdas deleted the jhabdas:jekyll-ga-generator branch Jan 24, 2017

@jhabdas

This comment has been minimized.

Show comment
Hide comment
@jhabdas

jhabdas Jan 24, 2017

Contributor

@pnn sorry I wasn't able to confirm this works with the latest version of Jekyll. But, if not, the added exposure ought to help make that shift occur. Appreciate the merge folks!

Contributor

jhabdas commented Jan 24, 2017

@pnn sorry I wasn't able to confirm this works with the latest version of Jekyll. But, if not, the added exposure ought to help make that shift occur. Appreciate the merge folks!

parkr added a commit that referenced this pull request Jan 26, 2017

Merge branch 'master' into release-3-4-0
* master: (39 commits)
  Update history to reflect merge of #5798 [ci skip]
  Update history to reflect merge of #5822 [ci skip]
  use logger.info
  run codeclimate after success
  Update history to reflect merge of #5819 [ci skip]
  Fixed inaccuracy in "Built-in permalink styles" docs [skip ci]
  Update history to reflect merge of #5802 [ci skip]
  Update history to reflect merge of #5811 [ci skip]
  Update history to reflect merge of #5690 [ci skip]
  Update history to reflect merge of #5815 [ci skip]
  Review CI pages
  Rework CI doc to include multiple providers.
  Update history to reflect merge of #5812 [ci skip]
  Add jekyll-ga plug-in
  Update configuration.md
  Add mention of classifier-reborn for LSI
  Update history to reflect merge of #5810 [ci skip]
  Got that diaper money?
  Added note about --blank flag
  Update history to reflect merge of #5797 [ci skip]
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment