Skip to content

Conversation

setchy
Copy link
Member

@setchy setchy commented Aug 16, 2025

Partially addresses #1997

Splits the notification filtering into a two-stage process

  • Base notification filters (type and reason)
  • Detailed notification filters (user type, handles, state)

This helps reduce detail enrichment api calls in certain circumstances.

@github-actions github-actions bot added the enhancement New feature or enhancement to existing functionality label Aug 16, 2025
@setchy
Copy link
Member Author

setchy commented Aug 16, 2025

@grahamj - this is part one of reducing api calls - hope you find it useful.

I'll follow-up with a separate to avoid fetching reviews etc if notifications have been filtered by their detailed state

Copy link

@setchy setchy changed the title feat(filters): split into two stages (base, detailed) to reduce wasted api calls feat(filters): split into two stages (base, detailed) to reduce api calls Aug 16, 2025
@setchy setchy merged commit 061e720 into main Aug 16, 2025
18 checks passed
@setchy setchy deleted the feat/reduce-api-calls-pt1 branch August 16, 2025 19:39
@github-actions github-actions bot added this to the Release 6.6.0 milestone Aug 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or enhancement to existing functionality
Development

Successfully merging this pull request may close these issues.

1 participant