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 DoubleToObjectConverter with GreaterThan and LessThan set #176

Merged
merged 2 commits into from
Aug 17, 2023
Merged

Fix DoubleToObjectConverter with GreaterThan and LessThan set #176

merged 2 commits into from
Aug 17, 2023

Conversation

davidegiacometti
Copy link
Contributor

Fixes

DoubleToObjectConverter not working as expected when GreaterThan and LessThan are both set

PR Type

  • Bugfix

What is the current behavior?

CommunityToolkit/WindowsCommunityToolkit#4854

What is the new behavior?

Fixed the wrong code in the converter and added tests.

PR Checklist

Please check if your PR fulfills the following requirements:

  • Created a feature/dev branch in your fork (vs. submitting directly from a commit on main)
  • Based off latest main branch of toolkit
  • Tests for the changes have been added (for bug fixes / features) (if applicable)
  • Header has been added to all new source files (run build/UpdateHeaders.bat)
  • Contains NO breaking changes

Other information

Copy link
Collaborator

@niels9001 niels9001 left a comment

Choose a reason for hiding this comment

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

Great! Thanks @davidegiacometti !

@niels9001 niels9001 merged commit b2dd749 into CommunityToolkit:main Aug 17, 2023
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.

2 participants