-
Notifications
You must be signed in to change notification settings - Fork 0
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
[#6603] home/blocks: add new video block with template #541
Conversation
058471b
to
18da8b5
Compare
86df7d1
to
eb43f1d
Compare
fd9cfb2
to
5d42f33
Compare
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.
Works nicely and looks great!
5d42f33
to
387cfd0
Compare
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.
Cool! Still looks good and works great. :)
We can merge now and delete the strings later or you delete them and then merge, whatever you prefer!
|
||
#~ msgid "For more information see" | ||
#~ msgstr "For more information see" | ||
|
||
#~ msgid "To protect your privacy this video can\\\\" | ||
#~ msgstr "To protect your privacy this video can\\\\" |
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.
The unused strings should be deleted from the files again.
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.
very cool!
docs: https://docs.wagtail.org/en/stable/advanced_topics/embeds.html