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

Improve typing around event emitter handlers #7816

Merged
merged 22 commits into from
Feb 22, 2022

Conversation

t3chguy
Copy link
Member

@t3chguy t3chguy commented Feb 16, 2022

Requires matrix-org/matrix-js-sdk#2180
Requires element-hq/element-web#21108


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

Preview: https://pr7816--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.

@t3chguy t3chguy added the T-Task Refactoring, enabling or disabling functionality, other engineering tasks label Feb 16, 2022
@t3chguy t3chguy changed the title Type js-sdk's event emitters Improve typing around event emitter handlers Feb 17, 2022
@t3chguy t3chguy marked this pull request as ready for review February 17, 2022 10:02
@t3chguy t3chguy requested a review from a team as a code owner February 17, 2022 10:02
Copy link
Member

@dbkr dbkr left a comment

Choose a reason for hiding this comment

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

so much ❤️ - thank you!

Copy link
Contributor

@germain-gg germain-gg left a comment

Choose a reason for hiding this comment

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

👏 👏

… t3chguy/typed-event-emitter

� Conflicts:
�	src/components/views/elements/AppTile.tsx
�	src/components/views/messages/MessageActionBar.tsx
�	src/components/views/rooms/EventTile.tsx
�	src/components/views/rooms/MessageComposer.tsx
�	src/stores/right-panel/RightPanelStore.ts
@t3chguy t3chguy merged commit 7fa01ff into develop Feb 22, 2022
@t3chguy t3chguy deleted the t3chguy/typed-event-emitter branch February 22, 2022 12:18
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.

3 participants