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

Convert /structures to TS #6787

Merged
merged 17 commits into from
Sep 16, 2021
Merged

Conversation

SimonBrandner
Copy link
Contributor

@SimonBrandner SimonBrandner commented Sep 12, 2021

Type: task


This leaves out the Group components and custom tag components for obvious reasons


This change is marked as an internal change (Task), so will not be included in the changelog.

Preview: https://6140dd36e5434e09bf306521--matrix-react-sdk.netlify.app
⚠️ Do you trust the author of this PR? Maybe this build will steal your keys or give you malware. Exercise caution. Use test accounts.

@github-actions github-actions bot added the T-Task Refactoring, enabling or disabling functionality, other engineering tasks label Sep 12, 2021
Signed-off-by: Šimon Brandner <simon.bra.ag@gmail.com>
Signed-off-by: Šimon Brandner <simon.bra.ag@gmail.com>
Signed-off-by: Šimon Brandner <simon.bra.ag@gmail.com>
Signed-off-by: Šimon Brandner <simon.bra.ag@gmail.com>
Signed-off-by: Šimon Brandner <simon.bra.ag@gmail.com>
@SimonBrandner SimonBrandner force-pushed the task/structures-ts branch 4 times, most recently from 316f3b2 to 55b2420 Compare September 12, 2021 08:11
Signed-off-by: Šimon Brandner <simon.bra.ag@gmail.com>
Signed-off-by: Šimon Brandner <simon.bra.ag@gmail.com>
Signed-off-by: Šimon Brandner <simon.bra.ag@gmail.com>
Copy link
Member

@t3chguy t3chguy left a comment

Choose a reason for hiding this comment

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

lgtm otherwise

src/components/structures/EmbeddedPage.tsx Outdated Show resolved Hide resolved
src/components/structures/EmbeddedPage.tsx Outdated Show resolved Hide resolved
src/components/structures/GenericErrorPage.tsx Outdated Show resolved Hide resolved
src/components/structures/IndicatorScrollbar.tsx Outdated Show resolved Hide resolved
src/components/structures/RoomStatusBar.tsx Outdated Show resolved Hide resolved
src/components/structures/ViewSource.tsx Outdated Show resolved Hide resolved
Signed-off-by: Šimon Brandner <simon.bra.ag@gmail.com>
Signed-off-by: Šimon Brandner <simon.bra.ag@gmail.com>
Signed-off-by: Šimon Brandner <simon.bra.ag@gmail.com>
Signed-off-by: Šimon Brandner <simon.bra.ag@gmail.com>
Copy link
Member

@t3chguy t3chguy left a comment

Choose a reason for hiding this comment

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

Great, thanks!

private unmounted = true;
private dispatcherRef: string;
private unmounted = false;
private dispatcherRef: string = null;
Copy link
Member

Choose a reason for hiding this comment

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

the = null isn't required but also not an issue

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
T-Task Refactoring, enabling or disabling functionality, other engineering tasks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants