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

Restructure API docs to avoid duplicate information #46

Open
towerofnix opened this issue Oct 29, 2018 · 0 comments
Open

Restructure API docs to avoid duplicate information #46

towerofnix opened this issue Oct 29, 2018 · 0 comments
Assignees

Comments

@towerofnix
Copy link
Owner

towerofnix commented Oct 29, 2018

This means two main things:

  • There will be a multitude of comment endpoints spread all throughout the API. Presently we only have endpoints for project pages, but eventually user and studio pages will be migrated to scratch-www and the new API; henceforth, we'll document the comment APIs for those pages as well. However, the APIs will almost certainly be exactly the same across profile, studio, and project comments. But in the current structure, we would end up copy-pasting the same information across each top-level API (/projects/<id>/comments, /users/<name>/comments, /studios/<id>/comments).

  • We currently duplicate a lot of information for the sake of consistency, but this isn't particularly fun to read. We should consider collecting information such as "returns an array of comment objects" across two endpoints into a single, more readable sentence. There is also a lot of "Limited to 40 results per request" spread about, which is a little dull to read.

I know that, across both these ideas, what I'm suggesting is to rewrite the docs as more of a guide than an API documentation, but I think it would be more useful in the format. I also suggest keeping an index page - a pure list of endpoints which link to the relevant section in "the guide" and can be ctrl-F'd (e.g. ctrl-F "delete" or "comment" or "user"). (Edit: This would be similar to the existing index page of the API (#34), but a complete list of endpoints rather than a table of contents.)

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

No branches or pull requests

1 participant