Conversation
|
/backport to stable26 |
|
/backport to stable25 |
| $user ??= $this->userSession->getUser(); | ||
|
|
||
| if ($user !== null) { | ||
| $userDefaultApps = explode(',', $this->config->getUserValue($user->getUID(), 'core', 'defaultapp')); |
There was a problem hiding this comment.
Does not seem to be updatable from the web UI but added in db86bea probably for admins to set with
occ user:setting admin core defaultapp filesThere was a problem hiding this comment.
Do we actually support users setting their own default app?
Also my main concern was more about the string itself. What does "defaultApp" refers too? There is no app named like that?
There was a problem hiding this comment.
Do we actually support users setting their own default app?
Can only be set by an admin from what I can see
Also my main concern was more about the string itself. What does "defaultApp" refers too? There is no app named like that?
Probably thought the string was the default fallback? 'defaultapp' is actually passed as the key argument for
Line 196 in 31f4f41
come-nc
left a comment
There was a problem hiding this comment.
All good apart from skjnldsv comments.
ca9e361 to
7c6ffd9
Compare
JuliaKirschenheuter
left a comment
There was a problem hiding this comment.
tested different scenarios and it works! Thank you
7c6ffd9 to
39fbb22
Compare
Signed-off-by: Christopher Ng <chrng8@gmail.com>
39fbb22 to
4f8e916
Compare
Summary
Add a descriptive label for the logo link
To be backported to
stable25for accessibilityTODO
spantranslated stringlogoUrlChecklist