Skip to content

Conversation

@kevinetienne
Copy link
Contributor

No description provided.

@kevinetienne kevinetienne force-pushed the add-api-endpoint-overview branch from 20103be to f1d95fb Compare March 19, 2015 16:25
@kevinetienne
Copy link
Contributor Author

@adam-incuna @meshy merge?

@adam-thomas
Copy link
Contributor

Unless the list is in some way wrong or incomplete (I don't know DUM well enough to confirm), this looks good :P

@meshy
Copy link
Contributor

meshy commented Mar 22, 2015

Might be worth explaining the modular nature of the urls module that provides these.

@adam-thomas
Copy link
Contributor

Travis hath exploded weirdly.

screenshot from 2015-03-23 09 46 50

docs/index.md Outdated
Copy link
Contributor

Choose a reason for hiding this comment

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

split ted

@meshy
Copy link
Contributor

meshy commented Mar 23, 2015

can the urls link to the views in the views.md file?

@kevinetienne kevinetienne force-pushed the add-api-endpoint-overview branch from e336b4a to 2301082 Compare March 24, 2015 11:03
@kevinetienne
Copy link
Contributor Author

@meshy views.md is not detailed well enough to link to it.

Ideally views.md should contain a paragraph introducing what they does in it.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 2301082 on add-api-endpoint-overview into ed8e647 on master.

@adam-thomas
Copy link
Contributor

This looks good to me, although see my earlier comment about not knowing DUM well enough to say for sure :)

jturnbull pushed a commit that referenced this pull request Sep 16, 2015
Add an overview of the API endpoints in the docs
@jturnbull jturnbull merged commit 400a1a4 into master Sep 16, 2015
@jturnbull jturnbull deleted the add-api-endpoint-overview branch September 16, 2015 08:49
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.

6 participants