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

Fix: make alternative addresses UX less confusing #4221

Merged
merged 10 commits into from
Mar 17, 2020
Merged

Fix: make alternative addresses UX less confusing #4221

merged 10 commits into from
Mar 17, 2020

Conversation

bwindels
Copy link
Contributor

@bwindels bwindels commented Mar 16, 2020 •
edited
Loading

@bwindels bwindels changed the title Bwindels/aliaslipstick Fix: make alternative addresses UX less confusing Mar 16, 2020
@bwindels
Copy link
Contributor Author

bwindels commented Mar 16, 2020 •
edited
Loading

Ahh, the e2ee tests are failing, will fix tomorrow. Build was just cancelled.
@bwindels bwindels added the X-Release-Blocker This affects the current release cycle and must be solved for a release to happen label Mar 16, 2020
@bwindels bwindels requested a review from a team March 16, 2020 17:31
@bwindels bwindels requested a review from nadonomy March 17, 2020 09:48
nadonomy
nadonomy approved these changes Mar 17, 2020
View reviewed changes
Copy link
Contributor

@nadonomy nadonomy left a comment

Choose a reason for hiding this comment

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

LGTM
jryans
jryans approved these changes Mar 17, 2020
View reviewed changes
Copy link
Collaborator

@jryans jryans left a comment

Choose a reason for hiding this comment

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

Looks good overall, thanks! 😄
bwindels and others added 2 commits March 17, 2020 10:43
@bwindels @jryans
Update src/components/views/room_settings/AliasSettings.js
1f4e928 
Co-Authored-By: J. Ryan Stinnett <jryans@gmail.com>
@bwindels
run i18n
0106612
@bwindels bwindels mentioned this pull request Mar 17, 2020
Merged
@bwindels bwindels merged commit e8c7754 into develop Mar 17, 2020
@luixxiul luixxiul mentioned this pull request May 1, 2023
Merged
3 tasks
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

3 participants

@bwindels @jryans @nadonomy