Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Email invites don't work #9816

Closed
5 tasks done
turt2live opened this issue May 23, 2019 · 8 comments
Closed
5 tasks done

Email invites don't work #9816

turt2live opened this issue May 23, 2019 · 8 comments
Assignees
Labels
A-3PIDs A-Invite P1 S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect X-Needs-Design

Comments

@turt2live
Copy link
Member

turt2live commented May 23, 2019

  1. Be in a room
  2. Send an email invite to someone to join the room
  3. They can't claim it

This affects /app as of writing.

The user clicks the link and is presented with a dialog-looking thing to create an account. They click this and are brought to the normal registration page. If the user happens to be logged in to an account already, they are just directed to the room rather than being told to verify their email address.

Not sure when this broke exactly, but it's not good.

Checkboxes for bugs identified here:

@turt2live
Copy link
Member Author

This broke in 1.1.0 as part of matrix-org/matrix-react-sdk#2925

The old version had this:
image

I'm taking it off my list, but leaving it as a fire for someone to fix. Given the age of the bug, I'm inclined to throw this into the "needs-design" bucket but have it fixed for the release after this week's (~2 weeks from now).

@turt2live turt2live removed their assignment May 23, 2019
@dbkr dbkr self-assigned this May 23, 2019
@dbkr
Copy link
Member

dbkr commented May 24, 2019

I've just gone through and tested all the different possible scenarios here. Here are the results. Bugs identified marked with [!!]:

not signed in, signup w/o email

  • get 'this invite wasn't sent to you' dialog
  • click 'try to join anyway'
  • joined

not signed in, signup with invited email

not signed in, signup with different email

  • no invite at all on either client [!!]

not signed in, sign in with account that has invited email (invite sent before email bound)

  • no invite [!!]

not signed in, sign in with account that doesn't have invited email

  • no invite [!!]

signed in with account that has invited email

  • (invite would get turned into a regular invite by being looked up)

signed in with account that doesn't have invited email

  • get 'this invite wasn't sent to you' dialog
  • click 'try to join anyway'
  • joined

@the-moog
Copy link

the-moog commented May 31, 2019

Still there in Riot-web 1.2.1 (RiotSDK 1.2.1) even though RiotSDK 1.1.1 'fixes' email registration.

@jryans
Copy link
Collaborator

jryans commented May 31, 2019

It's definitely known this is still an issue. It's being worked on.

@the-moog
Copy link

I got all exicited when I saw the 'fixed' comment in 1.1.1
Happy to help with any testing....

@dbkr
Copy link
Member

dbkr commented Jun 4, 2019

Yep, email registration and email invites are different things.

@dbkr
Copy link
Member

dbkr commented Jun 4, 2019

Another bug: if the HS has guest access disabled when you click on the email invite link, it just redirects to the welcome screen when it fails to register a guest user.

dbkr added a commit to matrix-org/matrix-react-sdk that referenced this issue Jun 6, 2019
Riot was always saying the email address that the invite was sent
to was not associated with your account.

Two fixes here:
 1. We mounted RoomPreviewBar with no invitedEmail prop and then
    changed the prop later, but RoomPreviewBar only checked for it
    on mount. Make sure we re-check when the props change.
 2. Pass oobData through RoomPreviewBar because we need to pass it
    to the RoomAvatar for 3pid invites.

element-hq/element-web#9816
dbkr added a commit to matrix-org/matrix-react-sdk that referenced this issue Jun 10, 2019
When the user was on an invite page and clicked the sign up/sign in
buttons, remember that invite so we can show it again after they're
done signing up/in.

element-hq/element-web#9816
@jryans
Copy link
Collaborator

jryans commented Jun 18, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-3PIDs A-Invite P1 S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect X-Needs-Design
Projects
None yet
Development

No branches or pull requests

4 participants