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

Add MemberInfo for 3pid invites and support revoking those invites #2843

Merged
merged 4 commits into from
Mar 29, 2019

Conversation

turt2live
Copy link
Member

@turt2live turt2live commented Mar 29, 2019

Fixes element-hq/element-web#625
Fixes element-hq/element-web#6411
Fixes element-hq/element-web#5490

Reviewer: This includes a minor semi-related change because it feels pointless to make it its own PR. It is however a dedicated commit to maintain sensible git history.

The panel is pretty bare bones, but it at least has some information on it:
image
image

@turt2live turt2live force-pushed the travis/cancel-3pid branch from 347f6e1 to 0258f61 Compare March 29, 2019 03:06
@turt2live turt2live requested a review from a team March 29, 2019 03:06
@jryans jryans self-requested a review March 29, 2019 09:39
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.

Thanks, this looks great overall! 😁

src/TextForEvent.js Outdated Show resolved Hide resolved
@@ -56,7 +56,7 @@ module.exports = React.createClass({
user.on("User._unstable_statusMessage", this._onStatusMessageCommitted);
},

componentWillUmount() {
componentWillUnmount() {
Copy link
Collaborator

Choose a reason for hiding this comment

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

😭

src/components/views/rooms/ThirdPartyMemberInfo.js Outdated Show resolved Hide resolved
@turt2live turt2live added the X-Release-Blocker This affects the current release cycle and must be solved for a release to happen label Mar 29, 2019
We just need to make sure they are structurally sound - actual validation is done by other parties.
@turt2live turt2live requested a review from jryans March 29, 2019 17:52
@turt2live
Copy link
Member Author

(re-review because changes are significant enough)

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.

Thanks for the extraction work! 😁 Seems clearer now.

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