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: Update link to i18n_filter plugin #6638

Merged
merged 2 commits into from Dec 17, 2017

Conversation

Projects
None yet
3 participants
@localheinz
Contributor

localheinz commented Dec 16, 2017

This PR

  • updates the link to the i18n_filter plugin in the plugin list

💁‍♂️ Since i18n_filter.rb has been removed with gacha/gacha.id.lv@004bffd, the current link is broken.

@@ -813,7 +813,6 @@ LESS.js files during generation.
- [Truncate HTML](https://github.com/MattHall/truncatehtml) by [Matt Hall](https://codebeef.com/): A Jekyll filter that truncates HTML while preserving markup structure.
- [Domain Name Filter by Lawrence Woodman](https://github.com/LawrenceWoodman/domain_name-liquid_filter): Filters the input text so that just the domain name is left.
- [Summarize Filter by Mathieu Arnold](https://gist.github.com/731597): Remove markup after a `<div id="extended">` tag.
- [i18n_filter](https://github.com/gacha/gacha.id.lv/blob/master/_plugins/i18n_filter.rb): Liquid filter to use I18n localization.

This comment has been minimized.

@localheinz

localheinz Dec 16, 2017

Contributor

Alternatively, we could remove it entirely.

@localheinz

localheinz Dec 16, 2017

Contributor

Alternatively, we could remove it entirely.

This comment has been minimized.

@DirtyF

DirtyF Dec 17, 2017

Member

Agreed, we'd better only list gems plugins from now on.

@DirtyF

DirtyF Dec 17, 2017

Member

Agreed, we'd better only list gems plugins from now on.

@localheinz localheinz changed the title from Fix: Remove i18n_filter from plugin list to Fix: Update link to i18n_filter plugin Dec 16, 2017

@DirtyF

This comment has been minimized.

Show comment
Hide comment
@DirtyF

DirtyF Dec 17, 2017

Member

Thanks @localheinz for keeping track on this ❤️

@jekyllbot: merge +docs

Member

DirtyF commented Dec 17, 2017

Thanks @localheinz for keeping track on this ❤️

@jekyllbot: merge +docs

@jekyllbot jekyllbot merged commit 16b9ce9 into jekyll:master Dec 17, 2017

1 of 3 checks passed

continuous-integration/appveyor/pr Waiting for AppVeyor build to complete
Details
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
WIP ready for review
Details

@localheinz localheinz deleted the localheinz:patch-2 branch Dec 17, 2017

@localheinz

This comment has been minimized.

Show comment
Hide comment
@localheinz

localheinz Dec 17, 2017

Contributor

Thank you, @DirtyF!

Contributor

localheinz commented Dec 17, 2017

Thank you, @DirtyF!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment