Skip to content

Conversation

@janepie
Copy link
Member

@janepie janepie commented Jun 18, 2025

Summary

Creates an OCS route to get a list of apps that are enabled for the currently logged-in user. No admin rights needed for this route.

(Needed for Context Agent to check the app's availability)

Checklist

@janepie janepie requested a review from a team as a code owner June 18, 2025 09:52
@janepie janepie requested review from Altahrim, come-nc and icewind1991 and removed request for a team June 18, 2025 09:52
@janepie janepie force-pushed the feat/add-user-enabled-apps-ocs branch from a3f85d1 to 1617845 Compare June 18, 2025 09:53
@janepie janepie force-pushed the feat/add-user-enabled-apps-ocs branch from 1617845 to 80b5d9d Compare June 18, 2025 10:00
@janepie janepie requested a review from provokateurin as a code owner June 18, 2025 10:07
@janepie janepie force-pushed the feat/add-user-enabled-apps-ocs branch from b4535e7 to e3166ba Compare June 18, 2025 10:13
@janepie janepie force-pushed the feat/add-user-enabled-apps-ocs branch 2 times, most recently from c149b94 to 5239c65 Compare June 18, 2025 11:33
@janepie
Copy link
Member Author

janepie commented Jun 18, 2025

Moved and adapted :) @julien-nc @provokateurin

@janepie janepie force-pushed the feat/add-user-enabled-apps-ocs branch from 5239c65 to baa2da5 Compare June 18, 2025 11:43
Copy link
Contributor

@oleksandr-nc oleksandr-nc left a comment

Choose a reason for hiding this comment

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

very nice

@janepie janepie force-pushed the feat/add-user-enabled-apps-ocs branch 2 times, most recently from 18e47aa to fe8aff5 Compare June 18, 2025 14:05
@janepie janepie force-pushed the feat/add-user-enabled-apps-ocs branch 2 times, most recently from 21711dd to cf3fd88 Compare June 18, 2025 15:20
janepie added 3 commits June 18, 2025 17:31
Signed-off-by: Jana Peper <jana.peper@nextcloud.com>
Signed-off-by: Jana Peper <jana.peper@nextcloud.com>
Signed-off-by: Jana Peper <jana.peper@nextcloud.com>
@janepie janepie force-pushed the feat/add-user-enabled-apps-ocs branch from cf3fd88 to 6f1e441 Compare June 18, 2025 15:31
@provokateurin provokateurin merged commit f05f79a into master Jun 18, 2025
199 of 203 checks passed
@provokateurin provokateurin deleted the feat/add-user-enabled-apps-ocs branch June 18, 2025 18:49
@janepie
Copy link
Member Author

janepie commented Jun 19, 2025

/backport to stable31

@janepie
Copy link
Member Author

janepie commented Jun 20, 2025

@susnux right, thanks for the reminder!

And because its backported: https://github.com/nextcloud/documentation/blob/master/developer_manual/app_publishing_maintenance/app_upgrade_guide/upgrade_to_31.rst#added-apis

Is this enough then or should I put it in both?

@susnux
Copy link
Contributor

susnux commented Jun 20, 2025

Hm good question, in theory 31 is enough

@nickvergessen
Copy link
Member

No one is rechecking old documentation after updating their app. So I would say add it in both

@skjnldsv skjnldsv mentioned this pull request Aug 19, 2025
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 enhancement pending documentation This pull request needs an associated documentation update

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants