-
Notifications
You must be signed in to change notification settings - Fork 83
Refactor label filtering logic #4982
Copy link
Copy link
Open
Labels
good first issueGood for newcomersGood for newcomers
Description
As per this comment, we currently allow filtering by label in two places (i.e. two endpoints), but the code to handle labels is duplicated.
In particular, the first implementation (here) handles input processing in the database layer, while the newer implementation has parameter splitting in the handler and inclusion/exclusion (i.e. label vs !label) in the service.
Refactor the older implementation into the second one, possibly reusing what is to be merged in the PR linked.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
good first issueGood for newcomersGood for newcomers