Skip to content

fix(NODE-3815): update Decimal128 constructor validation #476

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 5 commits into from
Dec 21, 2021
Merged

fix(NODE-3815): update Decimal128 constructor validation #476

merged 5 commits into from
Dec 21, 2021

Conversation

syz99
Copy link
Contributor

@syz99 syz99 commented Dec 16, 2021 •
edited
Loading

Description

What is changing?

Add new checkings/exceptions for invalid types

Is there new documentation needed for these changes?

n/a

Double check the following

  • Ran npm run lint script
  • Self-review completed using the steps outlined here
  • PR title follows the correct format: <type>(NODE-xxxx)<!>: <description>
  • Changes are covered by tests
  • New TODOs have a related JIRA ticket

@syz99 syz99 self-assigned this Dec 16, 2021
@syz99 syz99 requested review from baileympearson, dariakp and nbbeeken and removed request for dariakp and nbbeeken December 16, 2021 21:26
baileympearson
baileympearson previously requested changes Dec 16, 2021
View reviewed changes
@syz99 syz99 requested a review from baileympearson December 17, 2021 20:14
@nbbeeken nbbeeken requested review from durran and nbbeeken December 20, 2021 15:28
@nbbeeken nbbeeken added the Team Review Needs review from team label Dec 20, 2021
nbbeeken
nbbeeken previously approved these changes Dec 20, 2021
View reviewed changes
Copy link
Contributor

@nbbeeken nbbeeken left a comment

Choose a reason for hiding this comment

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

lgtm 👍

durran
durran requested changes Dec 20, 2021
View reviewed changes
@nbbeeken nbbeeken removed the request for review from baileympearson December 21, 2021 18:55
@nbbeeken nbbeeken dismissed baileympearson’s stale review December 21, 2021 18:56

Bailey OOO, comments addressed

@nbbeeken nbbeeken merged commit 95e8293 into master Dec 21, 2021
@nbbeeken nbbeeken deleted the NODE-3815/improve-error-message-for-improper-input-quoting branch December 21, 2021 18:56
addaleax added a commit to mongodb-js/compass that referenced this pull request Jan 12, 2022
@addaleax
fix(bson-transpilers): account for bson Decimal128 validation changes
14912d4 
Refs: mongodb/js-bson#476
Refs: https://jira.mongodb.org/browse/NODE-3815
@addaleax addaleax mentioned this pull request Jan 12, 2022
Merged
10 tasks
addaleax added a commit to mongodb-js/compass that referenced this pull request Jan 12, 2022
@addaleax
fix(bson-transpilers): account for bson Decimal128 validation changes (…
0cc8deb 
…#2679)

Refs: mongodb/js-bson#476
Refs: https://jira.mongodb.org/browse/NODE-3815
@github-actions github-actions bot mentioned this pull request May 7, 2024
Closed
@MrRaja23 MrRaja23 mentioned this pull request May 24, 2024
Open
@Isuf-Muca Isuf-Muca mentioned this pull request Aug 10, 2024
Open
@github-actions github-actions bot mentioned this pull request Jun 2, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@durran durran durran approved these changes

@baileympearson baileympearson baileympearson left review comments

+1 more reviewer

@nbbeeken nbbeeken nbbeeken approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@syz99 syz99

Labels

Team Review Needs review from team

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@syz99 @durran @nbbeeken @baileympearson