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

[MM-29651] Wait for async calls to finish before rendering anything for a couple billing screens #6780

Merged
merged 1 commit into from
Oct 14, 2020

Conversation

devinbinnie
Copy link
Member

@devinbinnie devinbinnie commented Oct 14, 2020

Summary

This PR fixes a flash that occurs on the Subscriptions and Payment Info screens when the information hasn't finished loading, but the component has rendered.

NOTE: I couldn't fix the billing history screen, see JIRA ticket for explanation.

Ticket Link

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

@devinbinnie devinbinnie added the 2: Dev Review Requires review by a core commiter label Oct 14, 2020
Copy link
Contributor

@alifarooq0 alifarooq0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@devinbinnie are you referring to the billing history screen?

@devinbinnie
Copy link
Member Author

@devinbinnie are you referring to the billing history screen?

Yes :D

@devinbinnie devinbinnie added 4: Reviews Complete All reviewers have approved the pull request and removed 2: Dev Review Requires review by a core commiter labels Oct 14, 2020
@devinbinnie devinbinnie merged commit c44e96d into mattermost:master Oct 14, 2020
@devinbinnie
Copy link
Member Author

/cherry-pick cloud-ga

@mattermod
Copy link
Contributor

Cherry pick is scheduled.

mattermost-build pushed a commit to mattermost-build/mattermost-webapp that referenced this pull request Oct 14, 2020
@mattermod mattermod added the CherryPick/Done Successfully cherry-picked to the quality or patch release tracked in the milestone label Oct 14, 2020
devinbinnie added a commit that referenced this pull request Oct 14, 2020
…or a couple billing screens (#6780) (#6782)

(cherry picked from commit c44e96d)

Co-authored-by: Devin Binnie <[email protected]>
@amyblais amyblais added Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation labels Oct 14, 2020
jfrerich pushed a commit that referenced this pull request Oct 23, 2020
calebroseland pushed a commit that referenced this pull request Oct 27, 2020
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 CherryPick/Done Successfully cherry-picked to the quality or patch release tracked in the milestone Docs/Not Needed Does not require documentation
Projects
None yet
5 participants