-
Notifications
You must be signed in to change notification settings - Fork 1.4k
DailyTransport: deprecated DailyUpdateRemoteParticipantsFrame #2742
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
DailyTransport: deprecated DailyUpdateRemoteParticipantsFrame #2742
Conversation
Codecov Report❌ Patch coverage is
🚀 New features to boost your workflow:
|
4899649 to
85ea1a3
Compare
CHANGELOG.md
Outdated
| future version. Create your own custom frame and use a custom processor to | ||
| handle it or use, for example, `on_after_push_frame` event instead in the | ||
| output transport. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wording nit
| future version. Create your own custom frame and use a custom processor to | |
| handle it or use, for example, `on_after_push_frame` event instead in the | |
| output transport. | |
| future version. Create your own custom frame and handle it in the | |
| `@transport.output().event_handler("on_after_push_frame")` event handler | |
| or a custom processor. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you. Done!
| warnings.simplefilter("always") | ||
| warnings.warn( | ||
| "DailyUpdateRemoteParticipantsFrame is deprecated and will be removed in a future version." | ||
| "Create your own custom frame and use a custom processor to handle it or use, for example, " |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same wording as above nit
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated.
85ea1a3 to
cf2c9d3
Compare
cf2c9d3 to
92d3be8
Compare
Please describe the changes in your PR. If it is addressing an issue, please reference that as well.
This PR deprecates
DailyUpdateRemoteParticipantsFrame. Developers can create their own custom frame and add a custom processor after the base output transport, or use on of the new events like, for example,on_after_push_framein the output transport.The main reason is to avoid having to support one frame per function and the corresponding handling of that frame.