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

No definition for ODM #37951

Closed
canberkvarli opened this issue Feb 4, 2025 · 4 comments · Fixed by #38011
Closed

No definition for ODM #37951

canberkvarli opened this issue Feb 4, 2025 · 4 comments · Fixed by #38011
Labels
Content:Learn:Express Learning area Express docs goal: clarity (Experimental label) Issues about unclear/confusing/inconcise content.

Comments

@canberkvarli
Copy link

MDN URL

https://developer.mozilla.org/en-US/docs/Learn_web_development/Extensions/Server-side/Express_Nodejs/mongoose

What specific section or headline is this issue about?

What is the best way to interact with a database?

What information was incorrect, unhelpful, or incomplete?

I believe there is a typo for Object Relational Mapper (ORM). Right on the next paragraph, it was defined as ODM which got myself confused.

What did you expect to see?

ORM (Object Relational Mapper)

Do you have any supporting links, references, or citations?

https://developer.mozilla.org/en-US/docs/Learn_web_development/Extensions/Server-side/Express_Nodejs/mongoose

Do you have anything more you want to share?

Image
@canberkvarli canberkvarli added the needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. label Feb 4, 2025
@github-actions github-actions bot added the Content:Learn Learning area docs label Feb 4, 2025
@hamishwillee
Copy link
Collaborator

This looks like a valid issue, was there a reason for closing it?

@hamishwillee hamishwillee reopened this Feb 4, 2025
@canberkvarli
Copy link
Author

Hey @hamishwillee!

Yea, looks like the ODM is kind of mentioned/explained in the following sections of that page although it is still misleading on the first glance...

@caugner
Copy link
Contributor

caugner commented Feb 5, 2025

Yes, it looks like ODM is valid, but the first occurrence should be expanded.

@caugner caugner added the goal: clarity (Experimental label) Issues about unclear/confusing/inconcise content. label Feb 5, 2025
@Josh-Cena Josh-Cena added Content:Learn:Express Learning area Express docs and removed Content:Learn Learning area docs needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. labels Feb 5, 2025
@Josh-Cena Josh-Cena changed the title Possible typo found in "docs/Learn_web_development/Extensions/Server-side/Express_Nodejs/mongoose" section No definition for ODM Feb 5, 2025
@hamishwillee
Copy link
Collaborator

Thanks - fixed by #38011 - essentially provide both acronyms so that it doesn't look like a typo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:Learn:Express Learning area Express docs goal: clarity (Experimental label) Issues about unclear/confusing/inconcise content.
Projects
None yet
4 participants