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

Implement /user/@userid:domain?action=chat #1006

Merged

Conversation

lukebarnard1
Copy link
Contributor

@lukebarnard1 lukebarnard1 commented Jun 2, 2017

This is a URL that can be used to start a chat with a user.

  • If the user is a guest, setMxId dialog will appear before anything and a defered action will cause ChatCreateOrReuseDialog to appear once they've logged in.
  • If the user is registered, they will not see the setMxId dialog.

fixes element-hq/element-web#4034

2017-06-02-114809_784x315_scrot
2017-06-02-114736_823x324_scrot
2017-06-02-114753_791x290_scrot

This is a URL that can be used to start a chat with a user.
 - If the user is a guest, setMxId dialog will appear before anything and a defered action will cause `ChatCreateOrReuseDialog` to appear once they've logged in.
 - If the user is registered, they will not see the setMxId dialog.

fixes element-hq/element-web#4034
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.

Otherwise lgtm

@@ -97,6 +97,7 @@ function createRoom(opts) {
// the room exists, causing things like
// https://github.com/vector-im/vector-web/issues/1813
if (opts.andView) {
console.info('And viewing');
Copy link
Member

Choose a reason for hiding this comment

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

did you mean to leave this in - it's a bit cryptic

Copy link
Contributor Author

Choose a reason for hiding this comment

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

nope, I'll remove it!

@dbkr dbkr assigned lukebarnard1 and unassigned dbkr Jun 2, 2017
@lukebarnard1 lukebarnard1 merged commit 953a573 into new-guest-access Jun 2, 2017
lukebarnard1 pushed a commit to element-hq/element-web that referenced this pull request Jun 2, 2017
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.

2 participants