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

MM-9711: Fix post attachment collapsing #901

Merged
merged 1 commit into from
Mar 7, 2018

Conversation

ccbrown
Copy link
Contributor

@ccbrown ccbrown commented Mar 7, 2018

Summary

Broken by #842. Fixed that, renamed the state variables to make it more apparent that their value is HTML, and verified that emojis still work as expected in attachments.

Ticket Link

https://mattermost.atlassian.net/browse/MM-9711

Checklist

  • Ran make check-style to check for style errors (required for all pull requests)

@ccbrown ccbrown added the 2: Dev Review Requires review by a core commiter label Mar 7, 2018
@jwilander
Copy link
Member

@ccbrown should this be against the release-4.8 branch?

@jasonblais jasonblais added this to the v4.8.0 milestone Mar 7, 2018
@hmhealey hmhealey added Awaiting Submitter Action Blocked on the author and removed 2: Dev Review Requires review by a core commiter labels Mar 7, 2018
@ccbrown ccbrown changed the base branch from master to release-4.8 March 7, 2018 18:26
@ccbrown ccbrown added 4: Reviews Complete All reviewers have approved the pull request and removed Awaiting Submitter Action Blocked on the author labels Mar 7, 2018
@GoldUniform GoldUniform merged commit 4a7b47e into mattermost:release-4.8 Mar 7, 2018
@amyblais amyblais added Changelog/Done Required changelog entry has been written Docs/Not Needed Does not require documentation labels Mar 7, 2018
@lindalumitchell lindalumitchell added the Tests/Done Release tests have been written label Mar 9, 2018
hmhealey pushed a commit that referenced this pull request Aug 28, 2020
#901)

* Channel loading state is determined by the chunks length
  * Dont filter or remove chunks if it is recent
hmhealey pushed a commit that referenced this pull request Mar 17, 2021
#901)

* Channel loading state is determined by the chunks length
  * Dont filter or remove chunks if it is recent
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/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.

7 participants