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

Manage guest accounts better on the welcome page for auth #2819

Closed
turt2live wants to merge 3 commits into from
Closed

Manage guest accounts better on the welcome page for auth #2819

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
5b2328b
Log out of the guest account if coming from the welcome page
turt2live Mar 23, 2019
a7f00e9
Switch the guest account over to the right homeserver after discovery
turt2live Mar 23, 2019
6b3bfd9
jsdoc for the linter
turt2live Mar 23, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions src/components/structures/MatrixChat.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1839,6 +1839,12 @@ export default React.createClass({
defaultIsUrl: isUrl,
loadingDefaultHomeserver: false,
});

// Change over the guest account, if one exists
if (MatrixClientPeg.get() && MatrixClientPeg.get().isGuest()) {
console.log("Discovered homeserver URLs - logging out current guest to use the homeserver");
Lifecycle.logout().then(() => this._loadSession());
Copy link
Copy Markdown
Collaborator

@jryans jryans Mar 26, 2019

Choose a reason for hiding this comment

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

Huh, so this will remake a new guest account once we discover the HS?

Copy link
Copy Markdown
Member Author

@turt2live turt2live Apr 11, 2019

Choose a reason for hiding this comment

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

yup, although it could probably do with a comment (or even better: not doing this and actually implementing the plan we talked about)

}
}
} catch (e) {
console.error(e);
Expand Down