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

Resources: add default_min_price/default_max_price check to free filt… #172

Merged

Conversation

danjacob-anders
Copy link

https://andersinno.atlassian.net/browse/TTVA-186

Adds check for min and max default prices to filter (max price included as min price can still be zero for some groups).

…er queries

If the default min or max price is not NULL and > 0 then we should include this
resource in the filtering logic.

Refs: TTVA-186
Copy link
Collaborator

@jopesy jopesy left a comment

Choose a reason for hiding this comment

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

Looks good. Nice job with the tests.

@danjacob-anders danjacob-anders merged commit a12cd56 into tre-varaamo-qa Dec 20, 2023
1 check passed
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