-
Notifications
You must be signed in to change notification settings - Fork 2.7k
MM-34366: Migrate 'components/post_view/post_list_virtualized' and associated tests to TypeScript #10426
Conversation
Hello @sinansonmez, Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for yet another PR to help us get through this migration!
I just have a couple small things to improve, but otherwise, this looks good
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @sinansonmez! LGTM 👍
I agree with Harrison's comments
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! This looks good to me now
/e2e-tests |
Successfully triggered e2e testing! |
/update-branch |
Successfully triggered e2e testing! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @sinansonmez
Tested, looks good to merge.
- Verified post list, new message line and toast, date separator, loading indicator etc. - working as before.
- E2E report looks good, no PR related failures.
Test server destroyed |
Summary
This PR migrates 'components/post_view/post_list_virtualized' and associated tests to TypeScript
Ticket Link
Fixes mattermost/mattermost#17289
JIRA: https://mattermost.atlassian.net/browse/MM-34366
Release Note