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 (graphql-middleware): User get Presenter status but can't stop external video #19550

Conversation

gustavotrott
Copy link
Collaborator

During the latest development call, as highlighted by @antobinary, an issue was identified in the system. When a user is granted presenter status, they encounter a problem where they are unable to stop external video playback, among other presenter-specific functionalities.

This Pull Request (PR) addresses and resolves this issue. The root cause of the problem was identified as follows: when a user becomes a presenter, the client-side application ceases various data subscriptions. This led to an issue where, if the Hasura backend responded with data related to these subscriptions, the client's middleware no longer recognized or handled this data, as the subscriptions had been stopped. The PR contains fixes to ensure that this problem does not occur when a user is assigned presenter status.

Copy link

This pull request has conflicts ☹
Please resolve those so we can review the pull request.
Thanks.

Copy link

sonarcloud bot commented Jan 31, 2024

Quality Gate Passed Quality Gate passed

The SonarCloud Quality Gate passed, but some issues were introduced.

16 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

Copy link

Automated tests Summary

All the CI tests have passed!

@TiagoJacobs TiagoJacobs merged commit 24acf01 into bigbluebutton:v3.0.x-release Jan 31, 2024
25 checks passed
@gustavotrott
Copy link
Collaborator Author

hey @ramonlsouza , can you confirm wether the bug is gone?

@ramonlsouza
Copy link
Member

closes #19545

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.

None yet

3 participants