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

[MM-29632] Added extra null checking for company address #6777

Merged
merged 2 commits into from
Oct 14, 2020

Conversation

devinbinnie
Copy link
Member

Summary

This PR fixes the case where the company_address field is null but the billing_address is not but still contains no data. A white screen would appear.

Ticket Link

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

@devinbinnie devinbinnie added the 2: Dev Review Requires review by a core commiter label Oct 14, 2020
@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 1dd306c into mattermost:master Oct 14, 2020
@amyblais amyblais added Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation labels 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
…6777)

* [MM-29632] Added extra null checking for company address

* Update components/admin_console/billing/company_info_display.tsx

Co-authored-by: Maria A Nunez <[email protected]>

Co-authored-by: Maria A Nunez <[email protected]>
(cherry picked from commit 1dd306c)
@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
* [MM-29632] Added extra null checking for company address

* Update components/admin_console/billing/company_info_display.tsx

Co-authored-by: Maria A Nunez <[email protected]>

Co-authored-by: Maria A Nunez <[email protected]>
(cherry picked from commit 1dd306c)

Co-authored-by: Devin Binnie <[email protected]>
jfrerich pushed a commit that referenced this pull request Oct 23, 2020
* [MM-29632] Added extra null checking for company address

* Update components/admin_console/billing/company_info_display.tsx

Co-authored-by: Maria A Nunez <[email protected]>

Co-authored-by: Maria A Nunez <[email protected]>
calebroseland pushed a commit that referenced this pull request Oct 27, 2020
* [MM-29632] Added extra null checking for company address

* Update components/admin_console/billing/company_info_display.tsx

Co-authored-by: Maria A Nunez <[email protected]>

Co-authored-by: Maria A Nunez <[email protected]>
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
Development

Successfully merging this pull request may close these issues.

5 participants