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

Development docs #206

Merged
merged 2 commits into from
Nov 12, 2015
Merged

Development docs #206

merged 2 commits into from
Nov 12, 2015

Conversation

lubomir
Copy link
Member

@lubomir lubomir commented Nov 10, 2015

Add general tips for creating new API. The goal is to have all the information summarized in one place, so that nothing is forgotten when new resources are added.

@bliuredhat
Copy link
Contributor

+1

1 similar comment
@xychu
Copy link
Contributor

xychu commented Nov 11, 2015

👍


#. Identify where to implement it: it can be part of existing application or
you can create a new application. If you want new application, use ::

Copy link
Contributor

Choose a reason for hiding this comment

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

want new application -> want a new application?

@ycheng-aa
Copy link
Contributor

Looks good.

@simozhan
Copy link
Contributor

Looks good to me


To describe available query filters, use ``%(FILTERS)s`` macro. This expands to
an unordered list with filter names and types of the value. The serializer can
be describe with ``%(SERIALIZER)s``, which expands to a code block with JSON
Copy link
Collaborator

Choose a reason for hiding this comment

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

describe -> described

@erichuanggit
Copy link
Collaborator

Generally looks good to me.

@lubomir lubomir merged commit 9f84d46 into master Nov 12, 2015
@lubomir lubomir deleted the development-docs branch November 12, 2015 07:40
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.

None yet

6 participants