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

Factor out MessageEvent.from() usage #9882

Merged
merged 4 commits into from
Jan 10, 2023

Conversation

turt2live
Copy link
Member

@turt2live turt2live commented Jan 9, 2023

The class/function is disappearing from the events-sdk, at least in this form.


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

The class/function is disappearing from the events-sdk, at least in this form.
@turt2live turt2live added the T-Task Refactoring, enabling or disabling functionality, other engineering tasks label Jan 9, 2023
The utility function is out of range of the calling code at runtime, for some reason.
@turt2live turt2live marked this pull request as ready for review January 10, 2023 00:23
@turt2live turt2live requested a review from a team as a code owner January 10, 2023 00:23
Copy link
Member

@andybalaam andybalaam 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!

@turt2live turt2live merged commit c09ca7b into develop Jan 10, 2023
@turt2live turt2live deleted the travis/extensible/messageevent-deprecation branch January 10, 2023 16:20
turt2live added a commit that referenced this pull request Jan 10, 2023
turt2live added a commit that referenced this pull request Jan 10, 2023
richvdh added a commit that referenced this pull request Jan 11, 2023
andybalaam pushed a commit to andybalaam/matrix-react-sdk that referenced this pull request Jan 12, 2023
* Factor out `MessageEvent.from()` usage

The class/function is disappearing from the events-sdk, at least in this form.

* Manually create contents for events used by cypress

The utility function is out of range of the calling code at runtime, for some reason.

* Run prettier

* Maybe this will fix the build
andybalaam pushed a commit to andybalaam/matrix-react-sdk that referenced this pull request Jan 12, 2023
andybalaam pushed a commit to andybalaam/matrix-react-sdk that referenced this pull request Jan 12, 2023
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