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 __truediv__ if result is Decimal #585

Merged
merged 2 commits into from Dec 29, 2020

Conversation

niklasb
Copy link
Contributor

@niklasb niklasb commented Dec 21, 2020

__truediv__ can return a decimal, in which case decimal_places cannot be set.

See #522 (comment)

@niklasb niklasb changed the title Fix truediv Fix __truediv__ if result is Decimal Dec 21, 2020
@Stranger6667
Copy link
Collaborator

Hi @niklasb !

Thank you for contributing! I will take a look during the day - failing tests are related to the latest changes in Django and I will fix them separately.

@Zagrebelin
Copy link

Hello @Stranger6667,

Did you fix the falling tests on django 3.2?
If no, can I suggest a quick fix for the new Django behavior? This bug is really bother me and I want to help you to fix it asap.
Zagrebelin@72b9625

@Stranger6667
Copy link
Collaborator

Hi @Zagrebelin ! Not yet, but thanks for the proposal! :)

@Stranger6667
Copy link
Collaborator

Thank you for your contribution!

@Stranger6667 Stranger6667 merged commit 564fb1c into django-money:master Dec 29, 2020
@Stranger6667
Copy link
Collaborator

Released in 1.2.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants