Skip to content

Conversation

@backportbot-libresign
Copy link

@backportbot-libresign backportbot-libresign bot commented Jan 1, 2026

Backport of #6295

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

- Remove static register() method using old IEventDispatcher pattern
- Add IAppManager dependency for isEnabledForUser() check
- Add Util::addScript() and Util::addStyle() calls in handle() method
- Provide initial state before loading scripts to fix menu disappearing issue
- Follow Talk/Spreed app pattern as reference implementation

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
- Change from LoadSidebarListener to TemplateLoader
- Use registerEventListener() instead of addServiceListener()
- Update imports to reflect new TemplateLoader location

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>

[skip ci]
- LoadSidebarListener functionality is now handled by TemplateLoader
- File no longer needed after refactoring to follow official Nextcloud pattern

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
@vitormattos vitormattos marked this pull request as ready for review January 1, 2026 18:22
@vitormattos vitormattos merged commit 728ddb9 into stable31 Jan 1, 2026
63 checks passed
@vitormattos vitormattos deleted the backport/6295/stable31 branch January 1, 2026 18:31
@github-project-automation github-project-automation bot moved this from 0. Needs triage to 4. to release in Roadmap Jan 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: 4. to release

Development

Successfully merging this pull request may close these issues.

2 participants