Skip to content

feat: notify about updates in main menu#1592

Merged
ShGKme merged 1 commit intonextcloud:mainfrom
max65482:feat_improved_update_notification
Feb 10, 2026
Merged

feat: notify about updates in main menu#1592
ShGKme merged 1 commit intonextcloud:mainfrom
max65482:feat_improved_update_notification

Conversation

@max65482
Copy link
Contributor

☑️ Resolves

🖼️ Screenshots

🏚️ Before 🏡 After
before (and OS notification) after

When a new update is available, the burger menu gets a little notification dot and a new menu entry for updating appears. This replaces the often overlooked OS notification.

@ShGKme
Copy link
Contributor

ShGKme commented Dec 16, 2025

Hi @max65482,

Thank you for your contribution, this looks good!
I'll leave some code-style/naming comments later today.

Just wanted to mention that we will be unable to include it in today's release because there are new strings to translate.

@max65482 max65482 force-pushed the feat_improved_update_notification branch from 9426f87 to ad20f2f Compare December 16, 2025 16:49
@max65482
Copy link
Contributor Author

Pushed a new commit and at least the linter should be happy now :)

@max65482
Copy link
Contributor Author

max65482 commented Feb 5, 2026

@ShGKme Could you review?

@ShGKme
Copy link
Contributor

ShGKme commented Feb 6, 2026

Hi @max65482,
I apologise it got lost during the Christmas time.
Let me have a look again.

@max65482 max65482 force-pushed the feat_improved_update_notification branch from 79b0ace to 777ab86 Compare February 8, 2026 06:22
@max65482
Copy link
Contributor Author

max65482 commented Feb 8, 2026

Thank you for your very constructive review! I implemented your suggestions.
The notification dot is now removed. Later we can add reusable notification components from @nextcloud/vue.

@max65482 max65482 requested a review from ShGKme February 8, 2026 06:31
@ShGKme ShGKme added the enhancement New feature or request label Feb 8, 2026
@ShGKme ShGKme added this to the 🪴 Next Release milestone Feb 8, 2026
Copy link
Contributor

@ShGKme ShGKme left a comment

Choose a reason for hiding this comment

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

Looks good, thank you.
Could you squash into a single commit?

Signed-off-by: Maximilian Martin <maximilian_martin@gmx.de>
@max65482 max65482 force-pushed the feat_improved_update_notification branch from 777ab86 to d4ff9a3 Compare February 9, 2026 17:42
@max65482
Copy link
Contributor Author

max65482 commented Feb 9, 2026

Done @ShGKme

@ShGKme ShGKme merged commit abd8809 into nextcloud:main Feb 10, 2026
11 checks passed
@ShGKme ShGKme mentioned this pull request Feb 20, 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update notification is too easy to miss

2 participants