-
Notifications
You must be signed in to change notification settings - Fork 113
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix open by default app and expose default app #2230
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, clearly having the default semantically explicit is better than just relying on the sorting of the app providers, which was definitely temporary.
At this point we could actually remove the logic to keep the app providers list ordered.
@glpatcern I updated the CS3apis, can we get this in? |
(did not merge yet because I'm not code owner - will check with @ishank011 also to update the |
[full-ci] update reva to get cs3org/reva#2230
needs: cs3org/cs3apis#157
needed because of owncloud/web#5970 (comment)
After this PR the
/app/list
endpoint will provide a default app (if configured):