Skip to content

fix(labrinth): fix filtering not working unless breakdown set to same field#6608

Open
penguinencounter wants to merge 2 commits into
modrinth:mainfrom
penguinencounter:fix-analytics-filters
Open

fix(labrinth): fix filtering not working unless breakdown set to same field#6608
penguinencounter wants to merge 2 commits into
modrinth:mainfrom
penguinencounter:fix-analytics-filters

Conversation

@penguinencounter

@penguinencounter penguinencounter commented Jul 4, 2026

Copy link
Copy Markdown

fixes #6607 plus some similar cases

a few notes:

  • project ID & dependent project ID work without changing their column enable logic
    • i did not check how they work, but they seem to work on prod so i didn't touch it
  • pretty much all other fields needed a || !metrics.filter_by.<COLUMN>.is_empty() added
  • user agent already had that, but was instead broken by not checking the correct thing when attaching the user agent data
  • cargo sqlx prepare deleted a bunch of files. we've put that into its own commit so it can be dropped with git reset --hard or git rebase -i relatively easily if it's not meant to be there
    • feel free to force-push the PR as needed

and a screenshot of it doing the right thing:
image

@modrinth-bot

Copy link
Copy Markdown
Member

Pull request changelog

App

Added

Changed

Deprecated

Removed

Fixed

Security

Website

Added

Changed

Deprecated

Removed

Fixed

Security

Hosting

Added

Changed

Deprecated

Removed

Fixed

Security

@penguinencounter penguinencounter force-pushed the fix-analytics-filters branch 9 times, most recently from ee71433 to 7690607 Compare July 5, 2026 02:10
@penguinencounter penguinencounter changed the title fix(labrinth): send columns relevant to analytics filters fix(labrinth): fix filtering not working unless breakdown set to same field Jul 6, 2026
@penguinencounter penguinencounter force-pushed the fix-analytics-filters branch 7 times, most recently from 7e6fc2a to 914f090 Compare July 8, 2026 06:13
@penguinencounter

Copy link
Copy Markdown
Author

would appreciate it if someone could look at this

@penguinencounter penguinencounter force-pushed the fix-analytics-filters branch from 914f090 to 3f3f6e0 Compare July 8, 2026 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Monetization data does not load unless "Monetization" breakdown is selected

2 participants