Skip to content
This repository has been archived by the owner on Jun 10, 2024. It is now read-only.

Handle Mongoose Validate error with try/catch #10102

Merged
merged 1 commit into from
Jan 5, 2024
Merged

Handle Mongoose Validate error with try/catch #10102

merged 1 commit into from
Jan 5, 2024

Conversation

dan-mba
Copy link
Member

@dan-mba dan-mba commented Jan 5, 2024 •
edited

Changes proposed

Re-update Mongoose to v8
Handle Model.Validate() errors with try / catch in all cases.

Check List (Check all the applicable boxes)

  • My code follows the code style of this project.
  • My change requires changes to the documentation.
  • I have updated the documentation accordingly.
  • All new and existing tests passed.
  • This PR does not contain plagiarized content.
  • The title of my pull request is a short description of the requested changes.

@github-actions github-actions bot added the dependencies Pull requests that update a dependency file label Jan 5, 2024
eddiejaoude
eddiejaoude approved these changes Jan 5, 2024
View reviewed changes
Copy link
Member

@eddiejaoude eddiejaoude left a comment

Choose a reason for hiding this comment

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

Thank you 👍

@eddiejaoude eddiejaoude merged commit 052f076 into main Jan 5, 2024
13 checks passed
@eddiejaoude eddiejaoude deleted the mongoose branch January 5, 2024 18:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@eddiejaoude eddiejaoude eddiejaoude approved these changes

Assignees
No one assigned
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@dan-mba @eddiejaoude