Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Additional fix for GetMemberForBoard returning nil,nil #3570

Merged
merged 2 commits into from
Aug 4, 2022
Merged

Additional fix for GetMemberForBoard returning nil,nil #3570

merged 2 commits into from
Aug 4, 2022

Conversation

sbishel
Copy link
Collaborator

@sbishel sbishel commented Aug 4, 2022

Summary

Additional fix for GetMemberForBoard returning nil,nil

See #3552

Ticket Link

NONE

@sbishel sbishel requested a review from wiggin77 August 4, 2022 19:53
@sbishel sbishel added the CherryPick/Approved Meant for the quality or patch release tracked in the milestone label Aug 4, 2022
@sbishel sbishel added this to the v7.2 milestone Aug 4, 2022
@sbishel sbishel merged commit 97818c3 into mattermost-community:main Aug 4, 2022
@mattermod
Copy link
Contributor

Cherry pick is scheduled.

mattermost-build pushed a commit to mattermost-build/focalboard that referenced this pull request Aug 4, 2022
@sbishel @mattermost-build
Additional fix for GetMemberForBoard returning nil,nil (mattermost-co…
8bf026e 
…mmunity#3570)

* fix panic

* remove else

(cherry picked from commit 97818c3)
@mattermost-build mattermost-build mentioned this pull request Aug 4, 2022
Merged
@mattermod mattermod added CherryPick/Done Successfully cherry-picked to the quality or patch release tracked in the milestone and removed CherryPick/Approved Meant for the quality or patch release tracked in the milestone labels Aug 4, 2022
@sbishel sbishel deleted the fix-panic-main branch August 4, 2022 20:30
sbishel added a commit that referenced this pull request Aug 4, 2022
@mattermost-build @sbishel
Additional fix for GetMemberForBoard returning nil,nil (#3570) (#3571)
c65da8c 
* fix panic

* remove else

(cherry picked from commit 97818c3)

Co-authored-by: Scott Bishel <scott.bishel@mattermost.com>
@wiggin77 wiggin77 mentioned this pull request Aug 8, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wiggin77 wiggin77 wiggin77 approved these changes

Assignees
No one assigned
Labels
CherryPick/Done Successfully cherry-picked to the quality or patch release tracked in the milestone
Projects
None yet
Milestone
v7.2
Development

Successfully merging this pull request may close these issues.
3 participants
@sbishel @mattermod @wiggin77