-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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 Server Side Ad manager stop #3611
Conversation
@joeyparrish can you review it? |
Were you able to resolve the issue mentioned in the comment?
Is this not happening any more? |
@joeyparrish The problem keeps happening. The problem here is that a part of the code is commented that causes other things to not be released correctly, so I have moved the comment to a lower level ( Note: this is a dependency of #3617 |
All tests passed! |
Thanks @joeyparrish ! |
This change causes the safe SS-DAI settings to be reset when a stream is unloaded.
This change causes the safe SS-DAI settings to be reset when a stream is unloaded.
This change causes the safe SS-DAI settings to be reset when a stream is unloaded.
This change causes the safe SS-DAI settings to be reset when a stream is unloaded.