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

After trying to join a room as a guest via RoomDirectory and getting rejected, i get stuck on a spinner #777

Closed
ara4n opened this issue Jan 25, 2016 · 4 comments
Labels
P1 S-Tolerable Low/no impact on users T-Defect

Comments

@ara4n
Copy link
Member

ara4n commented Jan 25, 2016

We shouldn't even allow onClick() to fire on RoomDirectory if there is no guest access or peek access.

@ara4n
Copy link
Member Author

ara4n commented Apr 13, 2016

@richvdh - this looks very close to where you just were?

@richvdh
Copy link
Member

richvdh commented Apr 13, 2016

Close, but different.

However, we no longer appear to get stuck on a spinner. We see a preview bar, with a "join" link; clicking on that results in "This room is private or inaccessible to guests. You may be able to join if you register." (I think this was improved in matrix-org/matrix-react-sdk#243).

There's still an argument that we shouldn't even let the user get that far - there is no point switching to the roomview if we know the join/peek is going to fail, but I think it makes this more a cosmetic bug.

@ara4n: wdyt?

@richvdh richvdh assigned ara4n and unassigned richvdh Apr 13, 2016
@ara4n ara4n added the S-Tolerable Low/no impact on users label Apr 13, 2016
@ara4n
Copy link
Member Author

ara4n commented Apr 13, 2016

agreed - let's just fire an ErrorDialog from RoomDirectory if the user tries to click into a room they can't join or peek.

@ara4n ara4n assigned richvdh and unassigned ara4n Apr 13, 2016
@richvdh richvdh removed their assignment Apr 13, 2016
@ara4n
Copy link
Member Author

ara4n commented Apr 13, 2016

fixed by b431de8

@ara4n ara4n closed this as completed Apr 13, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P1 S-Tolerable Low/no impact on users T-Defect
Projects
None yet
Development

No branches or pull requests

2 participants