Skip to content

[ENG-11866][ENG-11524][ENG-11871] Refactor notification campaign filters to use a dynamic query builder + Bug fixes - #11842

Merged
cslzchen merged 6 commits into
CenterForOpenScience:feature/project-enterfrom
Ostap-Zherebetskyi:feature/enter/filter_update
Jul 31, 2026
Merged

[ENG-11866][ENG-11524][ENG-11871] Refactor notification campaign filters to use a dynamic query builder + Bug fixes#11842
cslzchen merged 6 commits into
CenterForOpenScience:feature/project-enterfrom
Ostap-Zherebetskyi:feature/enter/filter_update

Conversation

@Ostap-Zherebetskyi

@Ostap-Zherebetskyi Ostap-Zherebetskyi commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator

Ticket

Purpose

Refactor notification campaign filters to use a dynamic query builder

In addition, fix URL encoding bug and enforce non-empty manual filter.

Changes

Side Effects

QE Notes

CE Notes

Documentation

@cslzchen cslzchen changed the title [] Refactor notification campaign filters to use a dynamic query builder [ENG-11866][ENG-11524] Refactor notification campaign filters to use a dynamic query builder + Fix URL encoding Jul 31, 2026
@cslzchen
cslzchen marked this pull request as ready for review July 31, 2026 15:06
@cslzchen cslzchen changed the title [ENG-11866][ENG-11524] Refactor notification campaign filters to use a dynamic query builder + Fix URL encoding [ENG-11866][ENG-11524] Refactor notification campaign filters to use a dynamic query builder + Bug fixes Jul 31, 2026
@cslzchen cslzchen changed the title [ENG-11866][ENG-11524] Refactor notification campaign filters to use a dynamic query builder + Bug fixes [ENG-11866][ENG-11524][ENG-11871] Refactor notification campaign filters to use a dynamic query builder + Bug fixes Jul 31, 2026

@cslzchen cslzchen left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@cslzchen
cslzchen merged commit 577be69 into CenterForOpenScience:feature/project-enter Jul 31, 2026
8 checks 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