-
Notifications
You must be signed in to change notification settings - Fork 118
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
chore: remove py dependency #314
Conversation
dddeb79
to
621d53c
Compare
source_attrs = {"src": video.get("video"), "type": "video/mp4"} | ||
return str(html.video(html.source(**source_attrs), **video_attrs)) | ||
return ( | ||
f'<video controls="" play-pause-on-click="" poster="{video.get("image")}" style="border:1px solid #e6e6e6;' |
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.
This is missing safe quoting off the inputs
621d53c
to
59c8665
Compare
Looks good! Please also add an entry to the changelog. 🙏 |
Please also add a shoutout to the person reporting the issue and to yourself for fixing it. 😊 |
8c2e865
to
7f92fb0
Compare
Not sure why the tests are failing. Can you take a look? |
7f92fb0
to
f96722b
Compare
Context path of the docker-compose file was incorrect. Fixed! |
f96722b
to
fa5d02c
Compare
@BeyondEvil After having a look at failing tests, the cause is the release of selenium package. Changes are linked to this issue #283 Can we merge this PR and proceed to the 4.0.2 release ? |
fa5d02c
to
965bd12
Compare
This fixes issue: #313