Skip to content

Disable cutting Option#454

Open
KeksBombe wants to merge 2 commits intoopencast-ilias:release_9from
KeksBombe:disable_cutting
Open

Disable cutting Option#454
KeksBombe wants to merge 2 commits intoopencast-ilias:release_9from
KeksBombe:disable_cutting

Conversation

@KeksBombe
Copy link
Copy Markdown
Contributor

This pull request introduces a new "Enable Cutting" feature, which allows administrators to control whether users can access the video cutting tool (editor) for events. The feature is configurable, disabled by default, and is reflected in both the UI and permissions logic.

@dagraf
Copy link
Copy Markdown

dagraf commented Oct 7, 2025

In meeting:

  • @chfsx will create a ReviewApp for testing. Then testing by @dagraf.
  • Possible other option: Do not configure the link to the editor.

@dagraf dagraf removed the To discuss Needs discussion with the community label Nov 4, 2025
@dagraf dagraf moved this from Backlog to In review in Opencast & Opencast PageComponent Nov 4, 2025
@dagraf
Copy link
Copy Markdown

dagraf commented Dec 2, 2025

@chfsx: Reminder.

@chfsx
Copy link
Copy Markdown
Contributor

chfsx commented Dec 4, 2025

Hi @dagraf here is the ReviewApp for this PR: http://pr-454.opencast.k8s.sr.solutions/ (currently root/review only)

@dagraf
Copy link
Copy Markdown

dagraf commented Jan 8, 2026

It seems to work. The setting is there in the sub-tab "Events" of the configuration "Settings" and does what it is said.

But I think the setting should be enabled by default. Otherwise, all adopters would need to change the configuration to get back the behavior they are used to. @KeksBombe: Can you please change this default setting?

@dagraf dagraf moved this from In review to In progress in Opencast & Opencast PageComponent Jan 8, 2026
@KeksBombe
Copy link
Copy Markdown
Contributor Author

Yes i can do that

@KeksBombe
Copy link
Copy Markdown
Contributor Author

Rebased on the current state of the release_9 and enabled by default to restore functionality

@dagraf
Copy link
Copy Markdown

dagraf commented Mar 16, 2026

@KeksBombe: Thx for changing the default setting.

@chfsx: Can you please review this PR and merge it, if everything is fine? Please take a close look if the default is implemented correctly. I did not test it. Thx!

@dagraf dagraf requested a review from chfsx March 16, 2026 17:04
@dagraf dagraf moved this from In progress to In review in Opencast & Opencast PageComponent Mar 16, 2026
@chfsx
Copy link
Copy Markdown
Contributor

chfsx commented May 5, 2026

Thanks a lot, I will release this in the next version. I will implement an äquivalent version for release_10 since we cannot pick this directly

@chfsx chfsx moved this from In review to Ready to merge in Opencast & Opencast PageComponent May 5, 2026
@chfsx chfsx added enhancement New feature or request ILIAS 9 Issue appears or may only appear on ILIAS 9. ILIAS 10 Currently based on http://10-redesign.opencast.k8s.sr.solutions/ labels May 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request ILIAS 9 Issue appears or may only appear on ILIAS 9. ILIAS 10 Currently based on http://10-redesign.opencast.k8s.sr.solutions/

Projects

Status: Ready to merge

Development

Successfully merging this pull request may close these issues.

3 participants