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/filter counts #406

Merged
merged 3 commits into from
Feb 28, 2025
Merged

fix/filter counts #406

merged 3 commits into from
Feb 28, 2025

Conversation

marcellmueller
Copy link
Contributor

@marcellmueller marcellmueller commented Feb 27, 2025

Fix for some of the new filter counts having different numbers than the results. Some of this was due me forgetting to filter rec resources by things like display_on_public_site, real world data being different than the dev fixtures as well as some duplicated data in recreation_access that was being imported.

I've deployed this to test and ensured the counts line up with the results.

Screenshot 2025-02-28 at 11 05 08 AM Screenshot 2025-02-28 at 11 04 54 AM Screenshot 2025-02-28 at 11 04 35 AM

@marcellmueller marcellmueller marked this pull request as ready for review February 28, 2025 19:06
@marcellmueller marcellmueller merged commit 6571971 into main Feb 28, 2025
32 checks passed
@marcellmueller marcellmueller deleted the fix/filter-counts branch February 28, 2025 21:20
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