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

VIDCS-3392: BG blur state is not saved for the waiting room #70

Merged
merged 14 commits into from
Feb 20, 2025

Conversation

behei-vonage
Copy link
Contributor

@behei-vonage behei-vonage commented Feb 13, 2025

What is this PR doing?

This PR fixes "BG blur state is not saved for the waiting room"

How should this be manually tested?

Follow the steps described in https://jira.vonage.com/browse/VIDCS-3392 to reproduce the issue.
Checkout this branch.
Follow the steps described in https://jira.vonage.com/browse/VIDCS-3392; notice that the issue is no longer present.

What are the relevant tickets?

A maintainer will add this ticket number.

Resolves VIDCS-3392

Checklist

[ ] Branch is based on develop (not main).
[ ] Resolves a Known Issue.
[ ] If yes, did you remove the item from the docs/KNOWN_ISSUES.md?
[ ] Resolves an item reported in Issues.
If yes, which issue? Issue Number?

@behei-vonage behei-vonage self-assigned this Feb 13, 2025
@behei-vonage behei-vonage added the do-not-review Do not review label Feb 13, 2025
@behei-vonage behei-vonage removed the do-not-review Do not review label Feb 13, 2025
maikthomas
maikthomas previously approved these changes Feb 14, 2025
Copy link
Collaborator

@maikthomas maikthomas left a comment

Choose a reason for hiding this comment

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

Approved but with a suggestion to avoid that disable line

@cpettet
Copy link
Contributor

cpettet commented Feb 18, 2025

Sorry, a follow-up comment. Do we save backgroundBlur to local storage anywhere for the waiting room? We might need to update the toggleBlur function, too.

Copy link
Contributor

@cpettet cpettet left a comment

Choose a reason for hiding this comment

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

Looks good so far! Just one thing I missed on first review. Let me know what you think!

@behei-vonage behei-vonage requested a review from cpettet February 18, 2025 20:44
Copy link
Contributor

@cpettet cpettet left a comment

Choose a reason for hiding this comment

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

Looks good so far! Just have a few comments/questions. Let me know what you think!

@behei-vonage behei-vonage requested a review from cpettet February 19, 2025 16:34
cpettet
cpettet previously approved these changes Feb 19, 2025
Copy link
Contributor

@cpettet cpettet left a comment

Choose a reason for hiding this comment

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

LGTM Great job! :shipit:

@behei-vonage behei-vonage requested a review from cpettet February 19, 2025 20:50
Copy link
Collaborator

@maikthomas maikthomas left a comment

Choose a reason for hiding this comment

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

Nice one!

@DeliaTok
Copy link
Collaborator

looks good!

@behei-vonage behei-vonage merged commit 7d09226 into develop Feb 20, 2025
7 checks passed
@behei-vonage behei-vonage deleted the behei-vonage/vidcs-3392-blur-waiting-room branch February 20, 2025 20:53
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.

4 participants