collective / wildcard.media

13 stars 12 forks source link

No way to make videos and audio optional #58

Closed pigeonflight closed 5 years ago

pigeonflight commented 5 years ago

current behaviour

When the video behaviour is enabled on a content type, it is not possible to save changes unless there is either a youtube url or an attached video (this is due to a registered invariant setting)

When the audio behaviour is enabled on a content type, it is not possible to save changes unless there is an attached audio.

This means that there is not way to support an optional audio or video attachment.

proposed changes to behaviour

There could be a global option which makes media attachements (audio/video) optional

pigeonflight commented 5 years ago

for now I've created a branch that makes video and audio optional. https://github.com/collective/wildcard.media/commit/2901c3cd4fa4af0415d46e8516c4ca6de1a71979

pigeonflight commented 5 years ago

I'm closing this as I have an alternative approach. My two aims are to make it easy to add a file field that supports streaming/seeking or the ability to easily skip to a section of a media file. When wildcard.media fits my use case it is great but there are scenarios that don't align as nicely (e.g. when I want the media fields to be optional). For this reason I'm proposing, at minimum, the ability support "seeking" out of the box on files with Plone.

See the suggestion here: https://github.com/plone/plone.formwidget.namedfile/issues/39 and here: #59