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

Add ways to manage addresses for Spaces #6151

Merged
merged 21 commits into from
Jun 22, 2021
Merged

Add ways to manage addresses for Spaces #6151

merged 21 commits into from
Jun 22, 2021

Conversation

t3chguy
Copy link
Member

@t3chguy t3chguy commented Jun 7, 2021

@t3chguy t3chguy requested a review from a team June 8, 2021 15:43
@t3chguy t3chguy marked this pull request as ready for review June 8, 2021 15:43
@t3chguy t3chguy requested a review from a team June 16, 2021 11:13
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.

generally looks fine - mostly human lint complaints and a few awkward state initializations.

src/components/views/dialogs/SpaceSettingsDialog.tsx Outdated Show resolved Hide resolved
src/components/views/elements/LabelledToggleSwitch.tsx Outdated Show resolved Hide resolved
src/components/views/elements/RoomAliasField.tsx Outdated Show resolved Hide resolved
src/components/views/elements/RoomAliasField.tsx Outdated Show resolved Hide resolved
src/components/views/elements/RoomAliasField.tsx Outdated Show resolved Hide resolved

// TODO: [REACT-WARNING] Move this to constructor
Copy link
Member

Choose a reason for hiding this comment

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

fwiw this was left behind because doing so caused crashes on user/room settings, and sometimes app-wide crashes. I presume it's been fixed, either intentionally or by accident over the last while?

Copy link
Member Author

Choose a reason for hiding this comment

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

Seems to be fine

src/components/views/spaces/SpaceSettingsGeneralTab.tsx Outdated Show resolved Hide resolved
src/components/views/spaces/SpaceSettingsVisibilityTab.tsx Outdated Show resolved Hide resolved
… t3chguy/fix/17244

� Conflicts:
�	src/components/views/room_settings/AliasSettings.tsx
@t3chguy
Copy link
Member Author

t3chguy commented Jun 22, 2021

E2E test passed, benchmark upload failed

@t3chguy t3chguy merged commit 50a5c03 into develop Jun 22, 2021
@t3chguy t3chguy deleted the t3chguy/fix/17244 branch June 22, 2021 21:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add a way to add an alias to a space
3 participants