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

[GH-12954] Migrate 'components/admin_console/color_setting.jsx' and associated tests to TypeScript #4080

Merged
merged 4 commits into from
Oct 30, 2019

Conversation

larkox
Copy link
Contributor

@larkox larkox commented Oct 28, 2019

Summary

Migrate 'components/admin_console/color_setting.jsx' and associated tests to TypeScript

Ticket Link

Fixes mattermost/mattermost#12954

Related Pull Requests

None

@hanzei hanzei added 2: Dev Review Requires review by a core commiter 3: QA Review Requires review by a QA tester labels Oct 29, 2019
@jespino jespino self-requested a review October 29, 2019 08:52
Copy link
Member

@jespino jespino 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 to me. Thanks @larkox

@jespino jespino requested a review from lindy65 October 29, 2019 08:56
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.

LGTM, thanks @larkox!

@devinbinnie devinbinnie removed the 2: Dev Review Requires review by a core commiter label Oct 29, 2019
@devinbinnie devinbinnie removed their assignment Oct 29, 2019
@lindy65 lindy65 added this to the v5.18.0 milestone Oct 30, 2019
Copy link
Contributor

@lindy65 lindy65 left a comment

Choose a reason for hiding this comment

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

Thanks @larkox!

@lindy65 lindy65 added QA Review Done Tests/Not Needed Does not require new release tests and removed 3: QA Review Requires review by a QA tester labels Oct 30, 2019
@jespino jespino added the 4: Reviews Complete All reviewers have approved the pull request label Oct 30, 2019
@jespino jespino merged commit 1270f3b into mattermost:master Oct 30, 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)
  ...
ashishmahi pushed a commit to ashishmahi/mattermost-webapp that referenced this pull request Oct 30, 2019
…ssociated tests to TypeScript (mattermost#4080)

* Migrate 'components/admin_console/color_setting.jsx' and test to Typescript. References also migrated.

* Fix lint issues.

* Updated snapshot
@amyblais amyblais added Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation labels Oct 30, 2019
brewsterbhg pushed a commit to brewsterbhg/mattermost-webapp that referenced this pull request Nov 11, 2019
…ssociated tests to TypeScript (mattermost#4080)

* Migrate 'components/admin_console/color_setting.jsx' and test to Typescript. References also migrated.

* Fix lint issues.

* Updated snapshot
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