Conversation
|
By analyzing the blame information on this pull request, we identified @bantu, @DeepDiver1975 and @icewind1991 to be potential reviewers |
There was a problem hiding this comment.
I assume that's not the first execution but only the interval. Mhm… Let me add then another fail safe somewhere.
LukasReschke
force-pushed
the
add-sso-for-updatechecker
branch
from
5c325f1 to
03a098b
Compare
apps/updatenotification/js/admin.js
Outdated
There was a problem hiding this comment.
Grmpfl… That thingy is a beast… If index.php is in the URL it fails because the URL generation generates something else 👿
LukasReschke
force-pushed
the
add-sso-for-updatechecker
branch
2 times, most recently
from
e0fa509 to
c35903f
Compare
Allows logging-in into the updater application by visiting the admin panel and pressing "Open updater".
LukasReschke
force-pushed
the
add-sso-for-updatechecker
branch
from
c35903f to
b9e3ed1
Compare
|
👍 works |
|
👍 |
|
@karlitschek @DeepDiver1975 can you comment on this app addition ? (just to be sure that you guys also agree on this being an app) |
|
different then what i had in mind but works for me :-) 👍 |
Add SSO for updater application
|
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
6 participants
Allows logging-in into the updater application by visiting the admin panel and pressing "Open updater".
Requires owncloud/updater#237
Follow-up of #22238
Fixes owncloud/updater#220