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

PLT-8103 Handle user updated event for profile image change for current user #323

Merged
merged 1 commit into from
Nov 21, 2017
Merged

PLT-8103 Handle user updated event for profile image change for current user #323

merged 1 commit into from
Nov 21, 2017

Conversation

jwilander
Copy link
Member

@jwilander jwilander commented Nov 20, 2017 •
edited
Loading

Summary

Handle user updated event for profile image change for current user. Once approved, I'll add it to mattermost-redux as well

Ticket Link

https://mattermost.atlassian.net/browse/PLT-8103

Checklist

@jwilander jwilander added the 2: Dev Review Requires review by a core commiter label Nov 20, 2017
enahum
enahum approved these changes Nov 20, 2017
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.

Can you add this to the WS in redux as well? or should we create another ticket for that?

@jwilander
Copy link
Member Author

@enahum You didn't read my summary :P I'm going to submit a redux fix as part of this ticket, just wanted this reviewed first

@saturninoabril saturninoabril added 4: Reviews Complete All reviewers have approved the pull request and removed 2: Dev Review Requires review by a core commiter labels Nov 20, 2017
@jwilander jwilander assigned jwilander and unassigned jwilander Nov 21, 2017
@jwilander jwilander merged commit cb7e99d into master Nov 21, 2017
@jwilander jwilander deleted the plt-8103 branch November 21, 2017 15:21
@jwilander jwilander mentioned this pull request Nov 24, 2017
Merged
@lindalumitchell lindalumitchell added the Tests/Done Release tests have been written label Nov 27, 2017
@esethna esethna added Changelog/Done Required changelog entry has been written Docs/Not Needed Does not require documentation labels Dec 4, 2017
hmhealey pushed a commit that referenced this pull request Aug 28, 2020
@enahum @hmhealey
Improve marking the channel as read (#323)
5a60a88 
* Improve marking the channel as read

* Feedback review

* Keep previous viewChannel action
hmhealey pushed a commit that referenced this pull request Mar 17, 2021
@enahum @jwilander
Improve marking the channel as read (#323)
cd2699d 
* Improve marking the channel as read

* Feedback review

* Keep previous viewChannel action
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@saturninoabril saturninoabril saturninoabril approved these changes

@enahum enahum enahum approved these changes

Assignees
No one assigned
Labels
4: Reviews Complete All reviewers have approved the pull request 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
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@jwilander @saturninoabril @enahum @esethna @lindalumitchell