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

Add name jump mark to renderer options & update section links to it #22

Open
wants to merge 1 commit into
base: source
Choose a base branch
from

Conversation

aCandidMind
Copy link

@aCandidMind aCandidMind commented Sep 11, 2017

This is a follow-up to #20 which applies to the actually used source branch.

Tested this by checking with middleman build and rackup.

@aCandidMind
Copy link
Author

aCandidMind commented Sep 11, 2017

Note that what's currently online on http://jsonapi-rb.org/guides/serialization/rendering.html with only having the rendering options doesn't seem to match what's in source/. Your new automatic deployment commits on master from some hours ago have put most content in the getting started section. With these changes also reflected in the source branch, this PR here wouldn't make sense, as only the rendering options section would be there. But the rendering page seems a bit out of context like that (the way it's currently online).

@beauby
Copy link
Member

beauby commented Sep 11, 2017

Hi @aCandidMind – thanks for pointing this out! I made the changes and the deploy in a branch and forgot to merge it into source. It should be in sync now!

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.

2 participants