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

API Routing would be a useful definition here. #9424

Merged
merged 5 commits into from
Oct 6, 2021
Merged

Conversation

TGillispie
Copy link
Contributor

Similar to application layer routing.

Summary

Adding API routing as it is used all over the place in API Development, (Express, Fastify, League/Route, etc.) Service layer or API routing is similar but different from client appliation routing.

Motivation

I'm building a list of (Restful API) vocabulary words for my team at NOAA NEFSC and am using MDN Web Docs as one of my primary resources. It would be nice to have a good definition of routing on the server.

Supporting details

Not sure.

Related issues

Metadata

  • Adds a new document
  • Rewrites (or significantly expands) a document
  • Fixes a typo, bug, or other error

Similar to application layer routing.
@TGillispie TGillispie requested a review from a team as a code owner October 1, 2021 16:59
@TGillispie TGillispie requested review from sideshowbarker and removed request for a team October 1, 2021 16:59
@github-actions github-actions bot added the Content:Glossary Glossary entries label Oct 1, 2021
@TGillispie TGillispie marked this pull request as draft October 1, 2021 17:20
@TGillispie TGillispie marked this pull request as ready for review October 1, 2021 17:28
@wbamberg
Copy link
Collaborator

wbamberg commented Oct 1, 2021

@TGillispie , I have a PR open at the moment to convert the Glossary into Markdown (#9384). Could I please ask that you resubmit this PR when that's merged?

@TGillispie
Copy link
Contributor Author

TGillispie commented Oct 1, 2021 via email

@wbamberg wbamberg closed this Oct 1, 2021
@wbamberg wbamberg reopened this Oct 5, 2021
@github-actions
Copy link
Contributor

github-actions bot commented Oct 5, 2021

Preview URLs

Flaws

None! 🎉

External URLs

URL: /en-US/docs/Glossary/routers
Title: Routers
on GitHub

No new external URLs

(this comment was updated 2021-10-06 04:19:17.181402)

Copy link
Collaborator

@wbamberg wbamberg left a comment

Choose a reason for hiding this comment

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

Thanks for this PR, @TGillispie ! Since my PR to convert the whole glossary has conflicts now anyway I thought we should try to merge this one before I try again.

I had a couple of comments. The first doesn't actually refer to any changes you've made and is more like copy-editing, the second is a bit more substantive and please let me know if I've destroyed the sense of what you were saying.

This page has a "See also" section listing examples of the existing definitions - it would be helpful I think to include examples of this one too.

files/en-us/glossary/routers/index.html Outdated Show resolved Hide resolved
files/en-us/glossary/routers/index.html Outdated Show resolved Hide resolved
Copy link
Collaborator

@wbamberg wbamberg left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution @TGillispie ! And it's great to know you're using MDN in NOAA!

(and yes, it can get pretty confusing. Please don't hesitate to ask if you want any help getting a contribution through.)

@wbamberg wbamberg merged commit 0b95a94 into mdn:main Oct 6, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 6, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Content:Glossary Glossary entries
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants