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

Draft PR for peeking with guest login #7

Open
wants to merge 4 commits into
base: peeking_unknown_rooms
Choose a base branch
from

Conversation

ashfame
Copy link
Member

@ashfame ashfame commented Feb 28, 2023

This PR (not meant for merge) highlights the differences needed in Hydrogen to make peeking with guest login work.

Run Automattic/chatrix#193 branch to run guest peeking

@ashfame ashfame self-assigned this Feb 28, 2023
@akirk
Copy link
Member

akirk commented Mar 1, 2023

Nice! So now the question is how doGuestLogin will be invoked.

@ashfame ashfame marked this pull request as ready for review March 3, 2023 15:56
@ashfame ashfame requested a review from psrpinto March 3, 2023 15:57
@ashfame ashfame added the upstream-contribution-candidate Supposed to go to upstream as a PR once base branch is merged label Mar 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
upstream-contribution-candidate Supposed to go to upstream as a PR once base branch is merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants