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

Updating redirect examples to 2.0 to halt confusion #36

Closed
wants to merge 4 commits into from

Conversation

dalanmiller
Copy link
Contributor

Hey @chipotle if you could merge this ASAP some of my sanity might be preserved.

I think a lot of people are discovering Horizon auth through this blog post and as a result are immediately doing it wrong since it's change with the release of 2.0.

If you could review and publish asap I would be in your debt.

@dalanmiller
Copy link
Contributor Author

Please wait 60 seconds for container to build the branch.
https://deploy.rethinkdb.com:32781

@dalanmiller
Copy link
Contributor Author

dalanmiller commented Sep 12, 2016

Gah wait on this, there's moar to fix.

Disregard, it's in the docs-archive folder.

@dalanmiller
Copy link
Contributor Author

@chipotle could you take a look at this?

@danielmewes
Copy link
Contributor

Should we add a note to the top of the post that states that the post was updated for Horizon 2.0? Otherwise people might look at the date of the post and assume that it uses Horizon 1.0.

@dalanmiller
Copy link
Contributor Author

@danielmewes to do that we'd need some sort of Jekyll tag which would make the message not show in the blog post preview list as I'd assume we'd want it at the top of the blog post.

@danielmewes
Copy link
Contributor

Hmm yeah. We could put it at the bottom instead?

@chipotle
Copy link

I don't think adding a note at the bottom is a bad idea, no.

@dalanmiller
Copy link
Contributor Author

The deploy has been updated, allow time for Jekyll to rebuild.
https://deploy.rethinkdb.com:32781

@dalanmiller
Copy link
Contributor Author

@dalanmiller
Copy link
Contributor Author

Ahh the latest post also doesn't have a <-- more --> which is rendering the whole blog post in the list view.

https://deploy.rethinkdb.com:32781/blog/

@dalanmiller
Copy link
Contributor Author

The deploy has been updated, allow time for Jekyll to rebuild.
https://deploy.rethinkdb.com:32781

@@ -514,6 +514,8 @@ to the [official documentation][docs].
* Horizon [permissions][permdoc] documentation
* [Complete source code][repo] of the Thinkmark demo

> Note that this post has been updated to use the Horizon 2.0 API

Choose a reason for hiding this comment

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

I know this is nitpicky, but I don't think this should be a block quote; it renders as if it's, well, a quote (indented, different color text, etc.). It'd be better if it were simply an italicized sentence (with a period at the end).

@chipotle
Copy link

Other than my nitpicky comment, this looks good.

@dalanmiller
Copy link
Contributor Author

Would you mind fixing it and deploying @chipotle?

2016-09-14 10:27 GMT-07:00 Watts Martin notifications@github.com:

Other than my nitpicky comment, this looks good.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#36 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA0sNrLfGNOWr60c-W0Np2AEJ6_qigmtks5qqC5wgaJpZM4J6vto
.

@dalanmiller
Copy link
Contributor Author

The deploy has been updated, allow time for Jekyll to rebuild.
https://deploy.rethinkdb.com:32781

@chipotle
Copy link

While Github didn't deign to automatically close this, this is merged as of ec721d0 and deployed.

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

Successfully merging this pull request may close these issues.

3 participants