-
-
Notifications
You must be signed in to change notification settings - Fork 827
Support changing your integration manager in the UI #3308
Conversation
f75855f
to
03d735f
Compare
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. I'll leave you to figure out the field widths etc. I think the reason it fits for you here but didn't for me is that my window is narrower and so there isn't enough space to the right of the settings dialog so it goes off the screen.
I'll adopt the use of the mixin and leave fixing it for element-hq/element-web#10554 (which I also plan to take on, just not here) |
This avoids us having to throw the entirety of MSC1957 into the queue, particularly when we're only using a third of the MSC.
Bleugh, then people will end up with this |
I'll just get the MSC through the process and start writing up the implementation we're missing. |
@dbkr done - now just not using vector-namespaced things |
Thanks - I've made the MSC slightly closer! |
Fixes element-hq/element-web#10161 (this is the last of the PRs in the series for this issue)
Known missing from this PR:
Based upon:
Refactor integration manager handling into a common place #3301mergedRead integration managers from account data (widgets) #3302mergedAdd UI in settings to change ID Server #3300mergedChangeset: 03d735f.changedGif: