Skip to content

Document the public API #99

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

Open
Asuza opened this issue Sep 6, 2018 · 5 comments
Open

Document the public API #99

Asuza opened this issue Sep 6, 2018 · 5 comments

Comments

@Asuza
Copy link
Contributor

Asuza commented Sep 6, 2018

The API, while undocumented, is currently public. It would be nice to have a page dedicated to explaining the various routes and payloads for developers to use if they so choose.

Perhaps Swagger or something similar would make sense for this.

Dredd could be used to test the documented API.

@scampiuk
Copy link
Contributor

scampiuk commented Oct 4, 2020

I would like to be assigned this issue please ❤️

@scampiuk
Copy link
Contributor

Just an update, this is coming along now
[-] documented most of the endpoints
[x] added route to view the documenation on the website
[ ] need to add tests

what I'm going to do I think is open a PR early on this, there are things I don't know and I will obviously have missed because, well, it's not my app :D

However, coming together.

Regarding the testing, I'm wondering if it's ok if I open in a different PR? Just stops 'one big pr' hitting

Cheers,

( sneak peak of the documentation viewer being served from inside the docker.. )

image

@Asuza
Copy link
Contributor Author

Asuza commented Oct 11, 2020

Absolutely. The primary goal of this issue was to document, not test. It's looking good!

@Asuza
Copy link
Contributor Author

Asuza commented Oct 23, 2020

@scampiuk, just wanted to check in since October is almost over. Can you make a PR for this?

@scampiuk
Copy link
Contributor

Yes sorry, I hate it when I sign up for things then forget I've got a job and two kids and stuff! :D I've got this week off work so i'll push the document up over the next 48 hrs

Asuza added a commit that referenced this issue Oct 31, 2020
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

2 participants