Skip to content

feat: add ocs route to get apps enabled for current user - #53569

Merged
provokateurin merged 3 commits into
masterfrom
feat/add-user-enabled-apps-ocs
Jun 18, 2025
Merged

feat: add ocs route to get apps enabled for current user#53569
provokateurin merged 3 commits into
masterfrom
feat/add-user-enabled-apps-ocs

Conversation

@janepie

Copy link
Copy Markdown
Member

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 ownerJune 18, 2025 09:52
@janepie
janepie requested review from Altahrim, come-nc and icewind1991 and removed request for a teamJune 18, 2025 09:52
@janepie
janepieforce-pushed the feat/add-user-enabled-apps-ocs branch from a3f85d1 to 1617845CompareJune 18, 2025 09:53
@janepie
janepieforce-pushed the feat/add-user-enabled-apps-ocs branch from 1617845 to 80b5d9dCompareJune 18, 2025 10:00
@janepie
janepieforce-pushed the feat/add-user-enabled-apps-ocs branch from b4535e7 to e3166baCompareJune 18, 2025 10:13
Comment threadapps/provisioning_api/appinfo/routes.php Outdated
Comment threadapps/provisioning_api/lib/Controller/AppsController.php Outdated
Comment threadapps/provisioning_api/lib/Controller/AppsController.php Outdated
Comment threadapps/provisioning_api/appinfo/routes.php Outdated
Comment threadapps/provisioning_api/lib/Controller/AppsController.php Outdated
Comment threadapps/provisioning_api/appinfo/routes.php Outdated
@janepie
janepieforce-pushed the feat/add-user-enabled-apps-ocs branch 2 times, most recently from c149b94 to 5239c65CompareJune 18, 2025 11:33
@janepie

Copy link
Copy Markdown
MemberAuthor

Moved and adapted :) @julien-nc@provokateurin

@janepie
janepieforce-pushed the feat/add-user-enabled-apps-ocs branch from 5239c65 to baa2da5CompareJune 18, 2025 11:43

@oleksandr-ncoleksandr-nc left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

very nice

Comment threadapps/provisioning_api/appinfo/routes.php Outdated
Comment threadapps/provisioning_api/lib/Controller/UsersController.php Outdated
@janepie
janepieforce-pushed the feat/add-user-enabled-apps-ocs branch 2 times, most recently from 18e47aa to fe8aff5CompareJune 18, 2025 14:05
@janepie
janepieforce-pushed the feat/add-user-enabled-apps-ocs branch 2 times, most recently from 21711dd to cf3fd88CompareJune 18, 2025 15:20
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
janepieforce-pushed the feat/add-user-enabled-apps-ocs branch from cf3fd88 to 6f1e441CompareJune 18, 2025 15:31
@provokateurin
provokateurin merged commit f05f79a into masterJun 18, 2025
@provokateurin
provokateurin deleted the feat/add-user-enabled-apps-ocs branch June 18, 2025 18:49
@janepie

Copy link
Copy Markdown
MemberAuthor

/backport to stable31

@janepie

Copy link
Copy Markdown
MemberAuthor

@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
Copy Markdown
Contributor

Hm good question, in theory 31 is enough

@nickvergessen

Copy link
Copy Markdown
Member

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

@skjnldsvskjnldsv mentioned this pull request Aug 19, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsenhancementpending documentationThis pull request needs an associated documentation update

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@janepie@susnux@nickvergessen@julien-nc@provokateurin@oleksandr-nc@tobiasKaminsky