-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Avoid guests from being stuck with numeric IDs #2947
Comments
This is a significant issue for me: I've been advocating and using Riot as a community organising hub for a makerspace, and the numeric IDs are actually breaking things really often.
|
@cathalgarvey thanks for the example failure mode. one temporary mitigation would be just to turn off guest access on your homeserver, which then forces users to pick a userid from the outset, and is close to the 2nd proposed solution on the original comment - but loses the "just get using the app" behaviour we have today. Alternatively we could prompt just for userid on first use, but doing so without specifying a password feels a bit weird and risky, hence wondering if we should so a full signup. "hmm." |
The right solution here is of course to let you migrate your user ID. |
So I think the solution we're going with now is
With the newly-suggested ROU -> PWLU-> Registered User flow (see element-hq/riot-meta#59) |
Fixed by ILAG me thinks |
I'm still having this issue — see element-hq/riot-meta#210 for desired functionality. Generic numbers for guests is dehumanizing! |
Guests are no longer used in riot-web so this is no longer an issue. |
When mixed with the autojoin option on synapse though, it's still an issue. See also matrix-org/synapse#3491 |
The issue summary says one of three things needs to be done, option 2 was taken :/ |
Except that at least with autojoin, guests can speak and participate in channels. To be fair, I'm not advocating that this be re-opened, just that there's an edge case. |
It sucks that guests can't escape numeric ids without losing their history. Need to either:
The text was updated successfully, but these errors were encountered: