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

Multiple videos for one Gloss #3

Open
stevecassidy opened this issue Feb 21, 2017 · 2 comments
Open

Multiple videos for one Gloss #3

stevecassidy opened this issue Feb 21, 2017 · 2 comments

Comments

@stevecassidy
Copy link
Contributor

@henrinie said:
We need to be able to upload multiple videos for one Gloss. We use GlossVideo.version to store the defined order of videos to show on gloss page. Users can arrange the videos to a desired order on the gloss page.
It seems that TaggedVideoManager would "mess" the version numbers for us if a new video is uploaded. On the other hand, it might not, the uploaded video would just show first and then it could be moved to where the user wants. Does TaggedVideoManager only update the version when a new video is added (it seems so)?

@stevecassidy
Copy link
Contributor Author

Currently the design is for a single video associated with one category/tag pair with versioning to allow 'undo' to recover the previously uploaded video.

What you describe is a different use case where you want multiple videos per Gloss. If you are currently subverting the version number to store your versions then I guess you could do this again here. However it would be better to have something that addressed your use more directly.

One option would be to append an index counter to the end of the 'tag' so store a video under 'Gloss'/'123_1' 'Gloss/'123_2' etc. You could then find these with a startswith filter.

Another option would be to add a third field into the model for an index number that would default to 0.

If we did this then you'd be able to push new versions of the Nth video for your gloss if you wanted to - not sure if that is a requirement for you.

@henrinie
Copy link
Member

I don't think that we need to have this versioning to be able to be able to 'undo' video uploads.
I will have to think about this.

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

No branches or pull requests

2 participants