Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TypeScript migration #6835

Closed
wants to merge 2 commits into from
Closed

TypeScript migration #6835

wants to merge 2 commits into from

Conversation

germain-gg
Copy link
Contributor

@germain-gg germain-gg commented Sep 18, 2021

  • Migrate PowerSelector to TypeScript
  • Migrate Pill to TypeScript

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

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

@germain-gg germain-gg added the T-Task Refactoring, enabling or disabling functionality, other engineering tasks label Sep 18, 2021
@germain-gg germain-gg requested a review from a team as a code owner September 18, 2021 07:18
@SimonBrandner
Copy link
Contributor

Covered by #6815 and #6398, though we might want to land the Pill.js conversion separately as my PR does some other changes

@germain-gg germain-gg closed this Sep 20, 2021
@germain-gg germain-gg deleted the gsouquet/ts-flight branch September 20, 2021 08:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-Task Refactoring, enabling or disabling functionality, other engineering tasks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants