Skip to content

feat(permissions): add DELETE_MEDIA permission to allow users to remove media from *arr services#2135

Open
0fflineuser wants to merge 9 commits into
seerr-team:developfrom
0fflineuser:develop
Open

feat(permissions): add DELETE_MEDIA permission to allow users to remove media from *arr services#2135
0fflineuser wants to merge 9 commits into
seerr-team:developfrom
0fflineuser:develop

Conversation

@0fflineuser

Copy link
Copy Markdown

… remove medias from arrs

Description

Screenshot (if UI-related)

To-Dos

  • Disclosed any use of AI (see our policy)
    I consulted AI to understand the codebase but the solution was fully authored manually by myself.
  • Successful build pnpm build
  • Translation keys pnpm i18n:extract

Issues Fixed or Closed

@0fflineuser 0fflineuser requested a review from a team as a code owner November 8, 2025 16:15
@0fflineuser 0fflineuser mentioned this pull request Nov 8, 2025
1 task
Comment thread .github/renovate/docker.json5 Outdated
Comment thread .github/renovate/pnpm.json5
Comment thread .vscode/settings.json
Comment thread src/components/ManageSlideOver/index.tsx
@0xSysR3ll 0xSysR3ll changed the title feat: new permission named DELETE_MEDIA so that we can allow users to… feat(permissions): add DELETE_MEDIA permission to allow users to remove media from *arr services Nov 11, 2025
Comment thread server/routes/media.ts
Comment thread src/components/ManageSlideOver/index.tsx Outdated
@github-actions github-actions Bot added the merge conflict Cannot merge due to merge conflicts label Nov 18, 2025
@github-actions

Copy link
Copy Markdown

This pull request has merge conflicts. Please resolve the conflicts so the PR can be successfully reviewed and merged.

@aserper

This comment was marked as off-topic.

@github-actions github-actions Bot removed the merge conflict Cannot merge due to merge conflicts label Dec 28, 2025
@stoneynutcase

stoneynutcase commented Jan 2, 2026

Copy link
Copy Markdown

Good first step! I'm running into the issue that having users being able to request but not delete media quickly blows up my used disk space...

I was looking into something similar but preferred DELETE_OWN_MEDIA (user would only be able to delete the media they requested). An extension on this could be that it needs to be approved.

@matieke

matieke commented Feb 12, 2026

Copy link
Copy Markdown

Any update on this?

@gauthier-th

Copy link
Copy Markdown
Member

Any update on this?

Merging of non-fix PRs is frozen until the Seerr merger is completed and the first version is released.

@github-actions github-actions Bot added the merge conflict Cannot merge due to merge conflicts label Feb 14, 2026
@github-actions

Copy link
Copy Markdown

This pull request has merge conflicts. Please resolve the conflicts so the PR can be successfully reviewed and merged.

@Zelzahn

Zelzahn commented Feb 26, 2026

Copy link
Copy Markdown

Really looking forward to what this PR would enable, so if I could help in any way let met know 🙂

@Tesurus

Tesurus commented Apr 20, 2026

Copy link
Copy Markdown

This would be great to not have to manually delete files for people

Comment on lines -9 to -14
radarrRoutes.get('/', (_req, res) => {
const settings = getSettings();

res.status(200).json(settings.radarr);
});

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why is this removed?

Comment on lines -9 to -14
sonarrRoutes.get('/', (_req, res) => {
const settings = getSettings();

res.status(200).json(settings.sonarr);
});

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why is this removed?

Comment thread server/routes/index.ts
const settings = getSettings();
return res.status(200).json(settings.sonarr);
}
);

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why is this moved here?

Comment thread .vscode/settings.json
"i18n-ally.localesPaths": [
"src/i18n/locale"
],
"i18n-ally.localesPaths": ["src/i18n/locale"],

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Unnecessary change

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merge conflict Cannot merge due to merge conflicts pending author's response

Projects

None yet

Development

Successfully merging this pull request may close these issues.

DELETE_MEDIA Permission

8 participants