Skip to content
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

Audio Component: Remove video tab from upload screen #7203

Closed
danielfmiranda opened this issue Aug 9, 2021 · 2 comments · Fixed by #7612
Closed

Audio Component: Remove video tab from upload screen #7203

danielfmiranda opened this issue Aug 9, 2021 · 2 comments · Fixed by #7612

Comments

@danielfmiranda
Copy link
Collaborator

Describe the request

When you are using the audio component in the cms, the upload file prompt shows two tabs Audio and Video. For this particular component we would like only the audio tab to appear to avoid confusion.

@KalobTaulien
Copy link
Contributor

I had thought this was for displaying panels on a Page and didn't realize this was for a stream block. Looks like wagtailmedia doesn't support this yet according to this issue: torchbox/wagtailmedia#133

We could fork the repo and apply the work we need done, but that's not typically something we like to do.

I'm going to move this to blocked.

CC @danielfmiranda @kristinashu

@KalobTaulien
Copy link
Contributor

Looks like support for this was added for StreamFields in torchbox/wagtailmedia#139

This is from the readme:

from wagtailmedia.blocks import AudioChooserBlock, VideoChooserBlock
class BlogPage(Page):
    # ...
    body = StreamField([
        # ... other block definitions
        ('audio', AudioChooserBlock(icon='media')),
        ('video', VideoChooserBlock(icon='media')),
    ])

I believe this needs a version bump in pypo from v 0.7.1 to the next version, but that's to come still.

I'm going to assign this to @danielfmiranda to implement. As far as I can tell, this will be a relatively easy change to make for the AudioBlock StructBlock.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants