Skip to content

fix: Cache enabled apps for user in appmanager - #64124

Merged
provokateurin merged 1 commit into
masterfrom
perf/cache-enabled-apps-per-user
Sep 10, 2026
Merged

fix: Cache enabled apps for user in appmanager#64124
provokateurin merged 1 commit into
masterfrom
perf/cache-enabled-apps-per-user

Conversation

@come-nc

@come-nc come-nc commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Summary

When some apps are enabled only for some groups this avoids recomputing
the list several times in the same request in some situations.

Checklist

AI (if applicable)

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

@come-nc come-nc self-assigned this Sep 8, 2026
@come-nc come-nc added the 2. developing Work in progress label Sep 8, 2026
When some apps are enabled only for some groups this avoids recomputing
 the list several times in the same request in some situations.

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
@come-nc
come-nc force-pushed the perf/cache-enabled-apps-per-user branch from dcef309 to 60ceab9 Compare September 10, 2026 10:06
@come-nc come-nc added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Sep 10, 2026
@come-nc come-nc added this to the Nextcloud 36 milestone Sep 10, 2026
@come-nc
come-nc marked this pull request as ready for review September 10, 2026 11:53
@come-nc
come-nc requested a review from a team as a code owner September 10, 2026 11:53
@come-nc
come-nc requested review from Altahrim, icewind1991, leftybournes and provokateurin and removed request for a team September 10, 2026 11:53
@provokateurin
provokateurin merged commit b355406 into master Sep 10, 2026
173 of 176 checks passed
@provokateurin
provokateurin deleted the perf/cache-enabled-apps-per-user branch September 10, 2026 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants