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

docs(guide+schematypes): add UUID to schematypes guide #13184

Merged
merged 4 commits into from Mar 21, 2023

Conversation

vkarpov15
Copy link
Collaborator

Fix #13032
Re: #13025

Summary

Add uuid to schematypes guide so we have some documentation to point to on how to work with uuids.

Examples

@vkarpov15 vkarpov15 requested a review from hasezoey March 17, 2023 15:04
Copy link
Collaborator

@hasezoey hasezoey left a comment

Choose a reason for hiding this comment

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

maybe also consider adding a note about mongoose.Types.UUID from #13186

also it would be good to note that not just uuid-v4 is supported, but all other versions and that mongoose does not do any checking that they are the same version (but does check that they match a pattern)

const UUID_FORMAT = /[0-9a-f]{8}-[0-9a-f]{4}-[0-9][0-9a-f]{3}-[89ab][0-9a-f]{3}-[0-9a-f]{12}/i;

@hasezoey hasezoey added the docs This issue is due to a mistake or omission in the mongoosejs.com documentation label Mar 17, 2023
@vkarpov15 vkarpov15 merged commit 80d29b3 into 6.x Mar 21, 2023
@vkarpov15 vkarpov15 deleted the vkarpov15/gh-13032 branch March 21, 2023 13:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs This issue is due to a mistake or omission in the mongoosejs.com documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants