Skip to content
This repository has been archived by the owner on Nov 21, 2018. It is now read-only.

en: update weekly medium link #306

Closed
wants to merge 2 commits into from
Closed

Conversation

rstacruz
Copy link

Btw, why should these live in the localized files? perhaps there should just be one place to update this at.

@diagramatics
Copy link
Contributor

Localized files are better IMO since different languages have different links to their own translated posts. I would rather have these updated automatically with RSS, but it adds an extra layer of complexity.

And what about changing the hyphen to – (en dash)? Minor typography issue, really.

@rstacruz
Copy link
Author

ah, fair enough!

@subfuzion
Copy link

I was about to submit a PR for the same thing when I saw this one already here. By the time this gets merged, it will be time for the March 27th update anyway. Hmm, makes one think there's got to be a better way... ;-)

@diagramatics
Copy link
Contributor

Guess this needs further discussion then. I made #307 discussing about using JS to read RSS feeds and show the latest post on the website.

@snostorm
Copy link
Contributor

Yup, out of date. Switching to the new discussion, we can accept a new PR. If you make one, please include the new index.html in the public directory as well (via gulp build). Saves us from having to do a followup commit.

@snostorm
Copy link
Contributor

Merged #301 instead of this -- I didn't see an update for the 27th up on Medium yet.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants