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

PLT-8684 Fixed DM links in channel sidebar #707

Merged
merged 2 commits into from
Feb 3, 2018
Merged

PLT-8684 Fixed DM links in channel sidebar #707

merged 2 commits into from
Feb 3, 2018

Conversation

hmhealey
Copy link
Member

@hmhealey hmhealey commented Feb 2, 2018

@hmhealey hmhealey added the 2: Dev Review Requires review by a core commiter label Feb 2, 2018
@hmhealey hmhealey added this to the v4.7.0 milestone Feb 2, 2018
enahum
enahum approved these changes Feb 2, 2018
View reviewed changes
Copy link
Contributor

@enahum enahum 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 but build failures, I think you need to update the snapshots or something like that

@hmhealey hmhealey merged commit d477103 into master Feb 3, 2018
@hmhealey hmhealey deleted the plt8684 branch February 3, 2018 03:30
@amyblais amyblais added Changelog/Done Required changelog entry has been written Docs/Not Needed Does not require documentation and removed Changelog/Done Required changelog entry has been written labels Feb 3, 2018
@lindalumitchell lindalumitchell added the Tests/Done Release tests have been written label Feb 6, 2018
@amyblais amyblais added the Changelog/Not Needed Does not require a changelog entry label Feb 17, 2018
hmhealey pushed a commit that referenced this pull request Aug 28, 2020
@chetanyakan @hmhealey
MM-12414 Remove redux hack (#707)
7ff2bd3
hmhealey pushed a commit that referenced this pull request Mar 17, 2021
@chetanyakan @jwilander
MM-12414 Remove redux hack (#707)
82466c7
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@stephenkiers stephenkiers stephenkiers approved these changes

@enahum enahum enahum approved these changes

@jespino jespino Awaiting requested review from jespino

Assignees
No one assigned
Labels
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/Done Release tests have been written
Projects
None yet
Milestone
v4.7.0
Development

Successfully merging this pull request may close these issues.
5 participants
@hmhealey @stephenkiers @enahum @lindalumitchell @amyblais