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

[MM-12582 & MM-12742] Remove PostActions removeReaction and addReaction and use the ones from redux #1864

Merged
merged 1 commit into from
Oct 11, 2018

Conversation

saturninoabril
Copy link
Member

@saturninoabril saturninoabril commented Oct 10, 2018

Summary

(1st PR)
Remove removeReaction and addReaction at PostActions and use the ones from redux.

Ticket Link

Jira ticket: MM-12582 & MM-12742

Checklist

  • Ran make check-style to check for style errors (required for all pull requests)
  • Ran make test to ensure unit and component tests passed
  • Added or updated unit tests (required for all new features)

@saturninoabril saturninoabril added the 2: Dev Review Requires review by a core commiter label Oct 10, 2018
@saturninoabril saturninoabril added this to the v5.5.0 milestone Oct 10, 2018
@hmhealey hmhealey modified the milestones: v5.5.0, v5.6.0 Oct 11, 2018
@hmhealey hmhealey removed the 2: Dev Review Requires review by a core commiter label Oct 11, 2018
@hmhealey hmhealey merged commit 64e8e2e into mattermost:master Oct 11, 2018
@saturninoabril saturninoabril deleted the MM-12582 branch October 11, 2018 18:22
@saturninoabril saturninoabril changed the title [MM-12582] Remove PostActions removeReaction and addReaction and use the ones from redux [MM-12582 & MM-12742] Remove PostActions removeReaction and addReaction and use the ones from redux Oct 16, 2018
@lindy65 lindy65 added the Tests/Done Release tests have been written label Oct 17, 2018
fincha pushed a commit to fincha/mattermost-webapp that referenced this pull request Oct 21, 2018
@amyblais amyblais added Changelog/Done Required changelog entry has been written Docs/Not Needed Does not require documentation labels Nov 17, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Changelog/Done Required changelog entry has been written Docs/Not Needed Does not require documentation Tests/Done Release tests have been written
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants