Skip to content

Draft PR for peeking with guest login#7

Open
ashfame wants to merge 4 commits into
peeking_unknown_roomsfrom
peeking_with_guest_login
Open

Draft PR for peeking with guest login#7
ashfame wants to merge 4 commits into
peeking_unknown_roomsfrom
peeking_with_guest_login

Conversation

@ashfame

@ashfame ashfame commented Feb 28, 2023

Copy link
Copy Markdown
Member

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

akirk commented Mar 1, 2023

Copy link
Copy Markdown
Member

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

@ashfame ashfame force-pushed the peeking_with_guest_login branch from efbc2f7 to 363a02b Compare March 3, 2023 15:52
@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
Comment thread src/matrix/sessioninfo/localstorage/SessionInfoStorage.ts Outdated
@ashfame ashfame force-pushed the peeking_with_guest_login branch from 363a02b to 2efd344 Compare March 7, 2023 14:30
@ashfame ashfame force-pushed the peeking_with_guest_login branch from 2efd344 to 36edfd4 Compare March 7, 2023 14:34
@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