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

GH-7782: Migrate channel_members_modal.jsx to be pure and use Redux #521

Merged
merged 2 commits into from
Jan 2, 2018
Merged

Conversation

skvale
Copy link
Contributor

@skvale skvale commented Dec 31, 2017

Summary

Refactor channel_members_modal.jsx to use redux

Ticket Link

mattermost/mattermost#7782

Checklist

  • Ran make check-style to check for style errors
  • Ran make test to ensure unit and component tests passed
  • Added or updated unit tests
  • Unable to get env: jest from eslint without b8f85e9

@grundleborg grundleborg added 1: PM Review Requires review by a product manager 2: Dev Review Requires review by a core commiter labels Jan 2, 2018
Copy link
Member

@jwilander jwilander left a comment

Choose a reason for hiding this comment

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

Awesome, thanks @skvale

@saturninoabril saturninoabril changed the title Issue 7782: Migrate channel_members_modal to redux GH-7782: Migrate channel_members_modal.jsx to be pure and use Redux Jan 2, 2018
Copy link
Member

@saturninoabril saturninoabril left a comment

Choose a reason for hiding this comment

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

Awesome, thanks for your contribution!

@saturninoabril saturninoabril merged commit 8a886c4 into mattermost:master Jan 2, 2018
@skvale skvale deleted the issue-7782 branch January 2, 2018 18:58
@lindalumitchell lindalumitchell added the Tests/Not Needed Does not require new release tests label Jan 2, 2018
@jasonblais jasonblais added Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation labels Jan 4, 2018
hmhealey pushed a commit that referenced this pull request Aug 28, 2020
* MM-10776: Adds support for new API endpoint to update team or channel member scheme roles. (#521)

* Disable fetch calls when offline (#522)

* Disable fetch calls when offline

* Move online check to doFetchWithResponse

* fix permission error on channel_converted event (in used by mobile RN) (#523)
hmhealey pushed a commit that referenced this pull request Aug 28, 2020
* MM-10776: Adds support for new API endpoint to update team or channel member scheme roles. (#521)

* Disable fetch calls when offline (#522)

* Disable fetch calls when offline

* Move online check to doFetchWithResponse

* fix permission error on channel_converted event (in used by mobile RN) (#523)

* warn about dropping backwards compatibility just once (#525)

* Safely trying to instantiate Set. (#527)

* Safely trying to instantiate Set.

* Added warning

* Removes console.warn.
hmhealey pushed a commit that referenced this pull request Mar 17, 2021
hmhealey pushed a commit that referenced this pull request Mar 17, 2021
* MM-10776: Adds support for new API endpoint to update team or channel member scheme roles. (#521)

* Disable fetch calls when offline (#522)

* Disable fetch calls when offline

* Move online check to doFetchWithResponse

* fix permission error on channel_converted event (in used by mobile RN) (#523)
hmhealey pushed a commit that referenced this pull request Mar 17, 2021
* MM-10776: Adds support for new API endpoint to update team or channel member scheme roles. (#521)

* Disable fetch calls when offline (#522)

* Disable fetch calls when offline

* Move online check to doFetchWithResponse

* fix permission error on channel_converted event (in used by mobile RN) (#523)

* warn about dropping backwards compatibility just once (#525)

* Safely trying to instantiate Set. (#527)

* Safely trying to instantiate Set.

* Added warning

* Removes console.warn.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
1: PM Review Requires review by a product manager 2: Dev Review Requires review by a core commiter Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation Tests/Not Needed Does not require new release tests
Projects
None yet
6 participants