-
-
Notifications
You must be signed in to change notification settings - Fork 426
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
fix(database): sum shows too many digits after decimal point #8833
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Your org has enabled the Graphite merge queue for merging into masterAdd the label “merge” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link. |
@OlegDev1 is attempting to deploy a commit to the toeverything Team on Vercel. A member of the Team first needs to authorize it. |
☁️ Nx Cloud ReportCI is running/has finished running commands for commit 0b0ec99. As they complete they will appear below. Click to see the status, the terminal output, and the build insights. 📂 See all runs for this CI Pipeline Execution ✅ Successfully ran 1 targetSent with 💌 from NxCloud. |
Merge activity
|
Thank you for starting the merging process of my PR! I have updated my branch to the latest changes If any changes / additions are required - please let me know, I am happy to help! |
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.
Could you sign the CLA for helping us land this PR? Thanks!
@doodlewind I have signed the CLI, thank you very much! Could you please provide some good first issues? |
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.
Landed the patch, thanks for your contribution!
…thing#8833) Co-authored-by: 3720 <zuozijian1994@gmail.com>
Fixes #8815
Fixes floating-point precision error, when calculating the sum of the column
I can write some unit tests, but unfortunately there are no unit tests in
data-view
folder. Any guidance will be highly appreciatedBefore the fix:
After the fix: