Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

while logging out ignore Session.logged_out as it is intentional #2058

Merged
merged 1 commit into from
Aug 7, 2018

Conversation

t3chguy
Copy link
Member

@t3chguy t3chguy commented Jul 15, 2018

and sometimes the server sent us 0-N 403's which would each yield a Warning Dialog before the logout completed and everything was unmounted/reset

Fixes element-hq/element-web#6911 and element-hq/element-web#5874

Signed-off-by: Michael Telatynski 7t3chguy@gmail.com

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
@bwindels bwindels requested a review from dbkr August 6, 2018 10:31
@bwindels
Copy link
Contributor

bwindels commented Aug 6, 2018

@dbkr could you have a quick check as well? Would be good to have this in as those dialogs on logout are pretty annoying.

@bwindels bwindels merged commit 07083e6 into develop Aug 7, 2018
@bwindels bwindels deleted the t3chguy/hide_logout_warning_whilst_logging_out branch August 7, 2018 08:31
Copy link
Member

@dbkr dbkr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only thing I would say is it'd be nice to do this without making Lifecycle stateful (and not with a module variable), but the only place we really have state is MatrixClientPeg, but that doesn't seem hugely better.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants