Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Fix RoomView re-mounting breaking peeking #5602

Merged
merged 1 commit into from
Feb 1, 2021
Merged

Conversation

t3chguy
Copy link
Member

@t3chguy t3chguy commented Jan 31, 2021

Because as of React 16 order of unmount & re-mount is undefined
this was causing the possibility of the willUnmount running after
the willMount of the replacement RoomView, upsetting the state
of the singleton inside the js-sdk.

Paired with matrix-org/matrix-js-sdk#1587

Because as of React 16 order of unmount & re-mount is undefined
this was causing the possibility of the unmount running after
the willMount of the replacement RoomView, upsetting the state
of the singleton inside the js-sdk.
@t3chguy t3chguy added the X-Release-Blocker This affects the current release cycle and must be solved for a release to happen label Jan 31, 2021
@t3chguy t3chguy requested a review from a team January 31, 2021 16:11
Copy link
Member

@turt2live turt2live left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please send to the release branch :)

@t3chguy t3chguy merged commit 72263a4 into develop Feb 1, 2021
@t3chguy t3chguy deleted the t3chguy/peeking branch February 1, 2021 12:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
X-Release-Blocker This affects the current release cycle and must be solved for a release to happen
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants