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.

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

fix(sidebar): user status reactivity bug #4939

Merged
merged 2 commits into from
Sep 20, 2022

Conversation

josephmcg
Copy link
Contributor

What this PR does 📖

Which issue(s) this PR fixes 🔨

Special notes for reviewers 🗒️

Additional comments 🎤

@josephmcg josephmcg linked an issue Sep 20, 2022 that may be closed by this pull request
@github-actions github-actions bot added the Missing Dev Review A Dev and a Dev Approver need to review the PR, then mark as Ready for QA label Sep 20, 2022
@stavares843 stavares843 added the QA tested One QA Person has tested - Needs QA Lead review still label Sep 20, 2022
Copy link
Contributor

@nathan-power nathan-power left a comment

Choose a reason for hiding this comment

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

LGTM!

@nathan-power nathan-power merged commit 26eb842 into dev Sep 20, 2022
@nathan-power nathan-power deleted the 4916-status-not-being-applied branch September 20, 2022 05:13
@github-actions github-actions bot removed Missing Dev Review A Dev and a Dev Approver need to review the PR, then mark as Ready for QA QA tested One QA Person has tested - Needs QA Lead review still labels Sep 20, 2022
josephmcg added a commit that referenced this pull request Sep 20, 2022
josephmcg added a commit that referenced this pull request Sep 20, 2022
@josephmcg josephmcg restored the 4916-status-not-being-applied branch September 20, 2022 05:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

status not being applied
3 participants