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

[MM-18988] Migrate components/recent_date to TypeScript #3883

Merged
merged 3 commits into from
Oct 25, 2019

Conversation

Pomyk
Copy link
Contributor

@Pomyk Pomyk commented Oct 6, 2019

Summary

Migrate components/recent_date and it's test to TypeScript

Ticket Link

Fixes mattermost/mattermost#12441

@hanzei hanzei added 2: Dev Review Requires review by a core commiter 3: QA Review Requires review by a QA tester labels Oct 7, 2019
@hanzei hanzei added this to the v5.18.0 milestone Oct 7, 2019
Copy link
Member

@devinbinnie devinbinnie left a comment

Choose a reason for hiding this comment

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

Looks great! Thanks @Pomyk!

Copy link
Member

@hmhealey hmhealey 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 submission, @Pomyk. Looks good to me!

]).isRequired,
}
type Props = {
value: DateSource;
Copy link
Member

Choose a reason for hiding this comment

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

Good to know that that type exists. I haven't looked at react-intl's typing information yet.

@hmhealey hmhealey removed the 2: Dev Review Requires review by a core commiter label Oct 8, 2019
@lindalumitchell lindalumitchell added QA Review Done and removed 3: QA Review Requires review by a QA tester labels Oct 15, 2019
@hanzei hanzei added the 4: Reviews Complete All reviewers have approved the pull request label Oct 25, 2019
@hanzei hanzei self-assigned this Oct 25, 2019
@hanzei hanzei merged commit b9d56dc into mattermost:master Oct 25, 2019
@amyblais amyblais added Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation labels Oct 28, 2019
iwataka added a commit to iwataka/mattermost-webapp that referenced this pull request Oct 30, 2019
…d-ui-testing-for-channel-reference-autocompletion

* upstream/master: (58 commits)
  MM-18541 - Updating header description css (mattermost#4048)
  [MM-19441] Fix the case of redirecting to an unavailable channel (mattermost#4070)
  [GH-12954] Migrate 'components/admin_console/color_setting.jsx' and associated tests to TypeScript (mattermost#4080)
  MM-18910 Migrate utils/emoticons to typescript (mattermost#3853)
  Mm 12447 Migrate 'components/get_link_modal.jsx' and associated tests to TypeScript (mattermost#3954)
  MM 17278 - Stop posts lists from scrolling when the emoji picker is open (mattermost#3860)
  MM-18157 - added loading screen to members modal (mattermost#3725)
  MM-19017 - Updating markdown images (mattermost#4047)
  MM-18956: error_pages components and tests to TS (mattermost#4050)
  Request team from API if invite_id is missing (mattermost#4072)
  MM-19678 - Restored setSelected in MultiSelectList component (mattermost#4033)
  [GH-12956] Migrate 'components/admin_console/banner.jsx' to TypeScript (mattermost#4075)
  [GH-12955] Migrate 'components/admin_console/highlight.jsx' to TypeScript (mattermost#4076)
  Change erroneous role='icon' to role='img' (mattermost#4051)
  [MM-18989] Migrate 'components/save_button.jsx' and associated tests to TypeScript (mattermost#4046)
  correct the height of the team sidebar when there's an announcement (mattermost#4073)
  Publishing storybook using circle-ci (mattermost#4037)
  [MM-18988] Migrate components/recent_date to TypeScript (mattermost#3883)
  MM-18629 - Remove default value for radio elements in interacti… (mattermost#4059)
  Replaced functionality in deprecated lifecycle methods (mattermost#3947)
  ...
brewsterbhg pushed a commit to brewsterbhg/mattermost-webapp that referenced this pull request Nov 11, 2019
@lindy65 lindy65 added the Tests/Not Needed Does not require new release tests label Nov 20, 2019
@hanzei hanzei removed their assignment Aug 14, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
4: Reviews Complete All reviewers have approved the pull request Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation Hacktoberfest QA Review Done Tests/Not Needed Does not require new release tests
Projects
None yet
7 participants