-
Notifications
You must be signed in to change notification settings - Fork 114
REST API Docs #297
REST API Docs #297
Conversation
stnguyen90
left a comment
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.
This is awesome and much-needed! I've added my comments
Co-authored-by: Steven <[email protected]>
Co-authored-by: Steven <[email protected]>
Co-authored-by: Steven <[email protected]>
Co-authored-by: Steven <[email protected]>
Co-authored-by: Steven <[email protected]>
Co-authored-by: Steven <[email protected]>
|
I also split the screenshot into many parts so it's easier to view ❤️ |
abnegate
left a comment
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.
Looking good. Is the syntax highlighting for HTTP working okay for you? I'm not seeing the request line highlighted, not sure if it's just me
Hmmmmmmmm, I think if you remove a space after the route, it highlights it differently but renders weird. I don't think the behavior is expected for prism. This is the most "normal" I can get it. Do we wanna spend some time digging into this? I'd need some help, I've no idea how the highlighting works. |
We should definitely get it working properly, I'll try have a look later today if I get some time |





What does this PR do?
Amends the new REST API docs.
What's still missing?
Test Plan
(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)
Related PRs and Issues
(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)
Have you read the Contributing Guidelines on issues?
(Write your answer here.)