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

PLT-8358 Properly show error bar after trying to edit a post after the allowed time #479

Merged
merged 4 commits into from
Dec 15, 2017

Conversation

hmhealey
Copy link
Member

The error bar for the edit post modal was no longer being displayed because the error it was trying to display didn't have a type.

I also fixed some related issues that Eric and I found where the modal would close randomly after the time period had past. This was due to some code that looked intentional as well as a selector that wasn't working correctly.

Ticket Link

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

@hmhealey hmhealey added the 2: Dev Review Requires review by a core commiter label Dec 15, 2017
@hmhealey hmhealey added this to the v4.5.0 milestone Dec 15, 2017
@hmhealey hmhealey changed the title Plt8358b PLT-8358 Properly show error bar after trying to edit a post after the allowed time Dec 15, 2017
@hmhealey hmhealey added 4: Reviews Complete All reviewers have approved the pull request and removed 2: Dev Review Requires review by a core commiter labels Dec 15, 2017
@crspeller crspeller merged commit 2e691a0 into release-4.5 Dec 15, 2017
@crspeller crspeller deleted the plt8358b branch December 15, 2017 20:41
@lindalumitchell lindalumitchell added the Tests/Done Release tests have been written label Dec 29, 2017
@jasonblais jasonblais added Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation labels Jan 3, 2018
hmhealey pushed a commit that referenced this pull request Aug 28, 2020
hmhealey pushed a commit that referenced this pull request Mar 17, 2021
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/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
5 participants