fossasia / eventyay-talk

eventyay speakers and talks component
Apache License 2.0
1.48k stars 47 forks source link

Fix speaker video view from video platform #151

Closed lcduong closed 1 month ago

lcduong commented 1 month ago

This PR closes/references issue speaker view in video component. It does so by return avartar_source and avartar_license as text

How has this been tested?

Checklist

Summary by Sourcery

This pull request addresses an issue with the speaker video view in the video component by modifying the return values of avatar_source and avatar_license to be text.

sourcery-ai[bot] commented 1 month ago

Reviewer's Guide by Sourcery

This pull request addresses an issue with the speaker view in the video component by modifying the return values of the get_avatar_source and get_avatar_license methods in the speaker.py serializer. The changes ensure that these methods return text values instead of calling them with an event parameter.

File-Level Changes

Files Changes
src/pretalx/api/serializers/speaker.py Updated methods to return avatar source and license as text values without event parameter.

Tips - Trigger a new Sourcery review by commenting `@sourcery-ai review` on the pull request. - Continue your discussion with Sourcery by replying directly to review comments. - You can change your review settings at any time by accessing your [dashboard](https://app.sourcery.ai): - Enable or disable the Sourcery-generated pull request summary or reviewer's guide; - Change the review language; - You can always [contact us](mailto:support@sourcery.ai) if you have any questions or feedback.