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

[FIX] stock: round w/ precision_rounding #163182

Closed
wants to merge 1 commit into from

Conversation

lvsz
Copy link
Contributor

@lvsz lvsz commented Apr 24, 2024

Versions

  • 17.0+

Issue

precision_rounding values were being passed incorrectly as precision_digits parameters.

Solution

Pass them as named precision_rounding parameters instead.

Enterprise branch: https://github.com/odoo/enterprise/pull/61387

@robodoo
Copy link
Contributor

robodoo commented Apr 24, 2024

@C3POdoo C3POdoo added the RD research & development, internal work label Apr 24, 2024
Versions
--------
- 17.0+

Issue
-----
`precision_rounding` values were being passed incorrectly as
`precision_digits` parameters.

Solution
--------
Pass them as named `precision_rounding` parameters instead.
@lvsz lvsz force-pushed the 17.0-precision_rounding-sile branch from ea882cb to 31fc4a5 Compare April 26, 2024 09:30
@lvsz lvsz marked this pull request as ready for review April 29, 2024 12:07
@C3POdoo C3POdoo requested a review from a team April 29, 2024 12:12
@ticodoo
Copy link
Contributor

ticodoo commented Apr 30, 2024

Good for me, but your ENT PR is for another team to review. I'm not sure if they will like that you changed the formatting 🤷

@robodoo r+

@robodoo
Copy link
Contributor

robodoo commented Apr 30, 2024

@lvsz @ticodoo linked pull request(s) odoo/enterprise#61387 not ready. Linked PRs are not staged until all of them are ready.

@robodoo robodoo closed this in b028287 May 2, 2024
willylohws pushed a commit to willylohws/odoo that referenced this pull request May 3, 2024
Versions
--------
- 17.0+

Issue
-----
`precision_rounding` values were being passed incorrectly as
`precision_digits` parameters.

Solution
--------
Pass them as named `precision_rounding` parameters instead.

closes odoo#163182

Related: odoo/enterprise#61387
Signed-off-by: Tiffany Chang (tic) <tic@odoo.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RD research & development, internal work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants