Skip to content

fix(core): make the app store link in the app menu configurable - #63860

Merged
pringelmann merged 1 commit into
masterfrom
fix/60495/appstore-link-visibility
Sep 1, 2026
Merged

fix(core): make the app store link in the app menu configurable#63860
pringelmann merged 1 commit into
masterfrom
fix/60495/appstore-link-visibility

Conversation

@pringelmann

@pringelmann pringelmann commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Summary

Non-admins get an "App store" tile linking to apps.nextcloud.com with no way to turn it off.
This introduces a new config key appstore_link_shown, with preset defaults:

Preset Link
none, private, family, club shown
shared, school, university, small, medium, large hidden

An explicitly set value always wins. Otherwise the link is hidden when appstoreenabled is off or a subscription is available, else the preset default applies. No preset means the link stays, so upgrades don't change.

This does also broaden appstoreenabled. It used to only gate installing from the store, now it hides the link too.

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI (tests, reviewed)

Signed-off-by: Peter Ringelmann <peter.ringelmann@nextcloud.com>
@pringelmann pringelmann added this to the Nextcloud 36 milestone Aug 31, 2026
@pringelmann pringelmann self-assigned this Aug 31, 2026
@pringelmann
pringelmann marked this pull request as ready for review August 31, 2026 14:22
@pringelmann
pringelmann requested review from a team as code owners August 31, 2026 14:23

@jancborchardt jancborchardt left a comment

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.

Nice refinement, as discussed :)

@jancborchardt jancborchardt moved this to 🏗️ At engineering in 🖍 Design team Aug 31, 2026
@pringelmann

Copy link
Copy Markdown
Contributor Author

/backport to stable35

@pringelmann

Copy link
Copy Markdown
Contributor Author

/backport to stable34

@pringelmann
pringelmann merged commit 33f1803 into master Sep 1, 2026
219 of 222 checks passed
@github-project-automation github-project-automation Bot moved this from 🏗️ At engineering to 🎉 Done in 🖍 Design team Sep 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: 🎉 Done

Development

Successfully merging this pull request may close these issues.

[Bug]: new app menu for Nextcloud 34: don't show "app store"-button for normal users

5 participants