-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
[stable24] Allow to tweak default scopes for accounts #32448
Conversation
Close #6582 Signed-off-by: Thomas Citharel <tcit@tcit.fr>
I suppose if we merge this we would need a new minor release? |
moving to 24.0.3 |
So seems like we will not merge this? then lets close it. |
I am fine to merge it, like @blizzz assigned it, let's just see that we get it merged before the next maintenance release, same for the 23 backport. |
Okay, lets @PVince81 merge it as we probably need to publish a 23.1.0 then... |
/** | ||
* The list of allowed scopes | ||
* | ||
* @since 25.0.0 |
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.
needs adjustment
@@ -1,9 +1,11 @@ | |||
<?php | |||
|
|||
/** | |||
* @copyright Copyright (c) 2016, ownCloud, Inc. |
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.
o.O
/** | ||
* The list of allowed properties | ||
* | ||
* @since 25.0.0 |
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.
needs adjustment
moving to 24.0.4… @tcitworld or @szaimen do you mind taking care of the requested adjustments? |
🏓 |
moving to 24.0.5. |
moving to 24.0.6 @AndyScherzinger @PVince81 if nobody is taking ownership of this PR I'd just close it next time 👀 |
It's more like a feature, no need to backport. |
backport of #31623