Skip to content

sample api reference template#1357

Closed
maryamsulemani97 wants to merge 7 commits intomasterfrom
test-api
Closed

sample api reference template#1357
maryamsulemani97 wants to merge 7 commits intomasterfrom
test-api

Conversation

@maryamsulemani97
Copy link
Copy Markdown
Contributor

No description provided.

@netlify
Copy link
Copy Markdown

netlify bot commented Jan 3, 2022

✔️ Deploy Preview for distracted-wiles-3edfc5 ready!

🔨 Explore the source changes: ceb34ea

🔍 Inspect the deploy log: https://app.netlify.com/sites/distracted-wiles-3edfc5/deploys/61d72b70cdbdeb0009db39f4

😎 Browse the preview: https://deploy-preview-1357--distracted-wiles-3edfc5.netlify.app

@maryamsulemani97 maryamsulemani97 linked an issue Jan 3, 2022 that may be closed by this pull request
@maryamsulemani97 maryamsulemani97 marked this pull request as ready for review January 25, 2022 06:29
@maryamsulemani97
Copy link
Copy Markdown
Contributor Author

The changes in this PR will be merged with #1535 or #1538 soon. Until then, I'd like to keep this here for reference.

Copy link
Copy Markdown
Contributor

@dichotommy dichotommy left a comment

Choose a reason for hiding this comment

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

  • I think there is value in the HTTP verb endpoint templates you created (GET, POST, DELETE etc), but in order for it to become a functional resource we can use every time we’re creating a new API reference page, I think ideally they should be combined into a single page/file.
  • Also, better if the template is not referring to an existing route (e.g. indexes) and is instead using something made up or comical, e.g. GET /foobar. This would make it easier to tell that they are templates and to for example use cmd+f to quickly find and replace all the template text (e.g. find and replace all foobar with keys, voila we have the skeleton of a keys API reference).
  • In the template file, we may want to include multiple examples of the same HTTP verb such as “Get one” + “Get all”, or “Delete one” + “Delete all”

@maryamsulemani97
Copy link
Copy Markdown
Contributor Author

Close once #1549 is approved

@maryamsulemani97 maryamsulemani97 marked this pull request as draft April 6, 2022 09:10
@guimachiavelli guimachiavelli deleted the test-api branch April 13, 2023 11:11
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.

API reference reformatting

2 participants