Skip to content

fix(navigation): show the unified inbox icon - #13475

Open
kesselb wants to merge 1 commit into
mainfrom
bug-warning-unified-mailbox-id
Open

fix(navigation): show the unified inbox icon#13475
kesselb wants to merge 1 commit into
mainfrom
bug-warning-unified-mailbox-id

Conversation

@kesselb

@kesselb kesselb commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Oversight from #13468

UNIFIED_INBOX_ID was imported but never exposed to the template, so the comparison always failed and the unified inbox rendered a generic folder icon.

馃 AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@kesselb kesselb self-assigned this Aug 12, 2026
@nextcloud-command nextcloud-command added the AI assisted This PR contains AI-assisted commits label Aug 12, 2026
@ChristophWurst
ChristophWurst requested a review from madbob August 12, 2026 10:59

@madbob madbob left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

You are right, sorry :-\

By the way, I also missed to remove now unused reference to UNIFIED_INBOX_ID in NavigationMailbox.

UNIFIED_INBOX_ID was imported but never exposed to the template, so the
comparison always failed and the unified inbox rendered a generic folder
icon.

Assisted-by: ClaudeCode:claude-opus-5
Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
@kesselb
kesselb force-pushed the bug-warning-unified-mailbox-id branch from 1d839bd to 5b76052 Compare August 12, 2026 11:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review AI assisted This PR contains AI-assisted commits bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants