-
-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
Change Links to not link to api.html
anymore
#12644
Conversation
7ea4463
to
f821a21
Compare
Updated to resolve conflicts from #12623 |
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.
LGTM minus one minor comment
docs/guide.md
Outdated
For example, we've defined a property `title` which will be cast to the | ||
[String](./api.html#schema-string-js) SchemaType and property `date` | ||
[String](./api/schema-string-js.html#schema-string-js) SchemaType and property `date` |
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 link is broken. It was also broken before. Should just link here: https://mongoosejs.com/docs/schematypes.html#strings
Summary
This PR changes links that pointed to
api.html
to a more proper place.Most commits are cherry-picked from #12223, though some changes have been made.
All Changes:
api.html
anymore but to more proper placescollection.d.ts
because there were invalid (even before the change of this PR)Changes from the commits in #12223: