-
Notifications
You must be signed in to change notification settings - Fork 181
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
Add Headless API #639
base: master
Are you sure you want to change the base?
Add Headless API #639
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You might wanna change my impl. a bit but:
Honestly, I can't remember my original reasoning so I'm gonna redo this to be all or nothing to create a simpler UX & DX. Thanks for bringing it up. |
Addressed all the feedback here and on the docs PR! 🚀 |
Reviewing the docs made me realize I have a more logistical comment for this PR: it exposes an This isn't a problem, but it means we need to consider it in versioning and should probably also document both sides. |
…into headless-api
Closes #638.
You can read the API docs here: https://github.com/boatbomber/rojo.space/blob/headless-api-docs/docs/headless-api.md