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

Purge import exclusions lists #10027

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

kopiro
Copy link

@kopiro kopiro commented May 15, 2024

Database Migration

NO

Description

Adding the possibility to clear (purge) the entire import exclusions list.

Screenshot (if UI related)

SCR-20240515-szvu
SCR-20240515-szwz

Todos

  • [] Reload the items after the XHR calls is done
  • Tests
  • Translation Keys (./src/NzbDrone.Core/Localization/Core/en.json)
  • Wiki Updates

Issues Fixed or Closed by this PR

@github-actions github-actions bot added Area: UI Issue is related to UI, should also add the issue to the new UI project, if it isn't a bug. Area: API Issue is related to the API Area: Import Lists Issue is related to Importing from Lists labels May 15, 2024
@mynameisbogdan
Copy link
Collaborator

I would push this to Sonarr first if you want in all arr apps.

Also personally I would go with the checkbox selection and removal with a bulk endpoint. And for all removal considering the existence of ClearBlocklistCommand and ClearLogCommand, I would suggest the same here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: API Issue is related to the API Area: Import Lists Issue is related to Importing from Lists Area: UI Issue is related to UI, should also add the issue to the new UI project, if it isn't a bug.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants