-
Notifications
You must be signed in to change notification settings - Fork 377
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
Add blog authors and other minor updates #228
Conversation
jwilander
commented
Jan 31, 2019
- Added the blog authors with links to community.mattermost.com and GitHub to bottom of blog posts
- Added blog authors on blog list page
- Removed blog tags and added more categories
The categories take me to a 404 error page, e.g. https://developers.mattermost.com/blog/npm-v6/categories/announcement Otherwise, great improvements here 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See note above regarding category links
Hmm looks like the categories were broken already, looking into it |
Odd they work fine locally for me |
@jasonblais I couldn't figure out why the categories weren't working so in the interim I just replaced them in the sidebar with links to other blog posts |
@@ -1,6 +1,12 @@ | |||
--- | |||
title: "Submitting Great PRs" | |||
slug: submitting-great-prs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not terribly worried, but I note that this will break any inbound links to this page. Is it easy to add a second slug? 0/5.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will add an alias for it
Chatted with Jason about the sidebar, it's not perfect but we're ok with it for now |