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

Document JSON API implementation defs and progress in class #1482

Merged
merged 1 commit into from
Mar 13, 2016

Conversation

bf4
Copy link
Member

@bf4 bf4 commented Jan 29, 2016

Update 2016-03-08

Added specs as comments in code... call to attention spec boundaries versus method boundaries

original description

I was thinking of breaking down the parts of the JSON API
spec that we need to implement into a kind of interface like
this so that we could highlight the areas that are complete or incomplete.

Refs:

# ☐ toplevel_errors (required)
# ☐ toplevel_meta
# ☑ toplevel_jsonapi
def failure
Copy link
Member Author

Choose a reason for hiding this comment

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

#
# description:
# An object describing the server's implementation
def toplevel_jsonapi
Copy link
Member Author

Choose a reason for hiding this comment

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

@bf4 bf4 changed the title [RFC] Proposed JSON API object skeleton JSON API object skeleton to build existing and future implementations on Feb 29, 2016
error,
error
]
end
Copy link
Member Author

Choose a reason for hiding this comment

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

these should be comments with # @example and the implementation is raise NotImplementedError

@bf4 bf4 force-pushed the jsonapify branch 2 times, most recently from 9e10660 to ab6af9d Compare March 9, 2016 05:58
@bf4 bf4 changed the title JSON API object skeleton to build existing and future implementations on Document JSON API implementation defs and progress in class Mar 9, 2016
bf4 added a commit that referenced this pull request Mar 13, 2016
Document JSON API implementation defs and progress in class
@bf4 bf4 merged commit fcd394a into rails-api:master Mar 13, 2016
@bf4 bf4 deleted the jsonapify branch March 13, 2016 07:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant