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

[wishlist] one or more url archive link fields per slot, for linking to archives of that performance #20

Closed
cleary opened this issue Feb 6, 2022 · 6 comments · Fixed by #23
Assignees
Projects

Comments

@cleary
Copy link

cleary commented Feb 6, 2022

we're currently brainstorming some post-event display stuff in muxy-frontend, and thought it'd be handy to link to the archived performance (on youtube.com/archive.org) from the performer card on night.tidalcycles.org (for example)

The place to keep that info of course is here, and sfradkin can add/update the links via the api when doing the video processing/upload

@munshkr
Copy link
Owner

munshkr commented Feb 8, 2022

Sounds good! So it would be a single URL for each Stream, to be updated post-event, right?

@sfradkin
Copy link

sfradkin commented Feb 8, 2022

Multiple URLs

@cleary
Copy link
Author

cleary commented Feb 8, 2022

Sounds good! So it would be a single URL for each Stream, to be updated post-event, right?

Multiple urls, we're currently archiving each individual stream to YouTube.com and archive.org

@munshkr munshkr added this to To do in Muxy Mar 16, 2022
@raphaelbastide
Copy link

I like the idea of having archived video URLs to each slot. Maybe the EDIT button could be removed in the same time.

@munshkr munshkr moved this from To do to In progress in Muxy May 30, 2022
@munshkr munshkr self-assigned this May 30, 2022
@munshkr munshkr mentioned this issue May 30, 2022
5 tasks
Muxy automation moved this from In progress to Done May 30, 2022
munshkr added a commit that referenced this issue May 30, 2022
* Define `StreamArchiveURL` model and create migration
* Register `StreamArchiveURL` admin and add inline to Stream admin
* Add stream archive URLs as nested serializer on `StreamSerializer`
* Set fk related name to `archive_urls`
* Make archive urls nested serializer writable on `StreamSerializer`
* Make `archive_urls` not required on serializer
* Test partial update (PATCH) on Stream, to update `archive_urls` of a Stream

Closes #20
@munshkr
Copy link
Owner

munshkr commented May 30, 2022

Sorry it took so long. I've added Archive URLs to the Stream model. These can be added either via admin or API. You can check out the API documentation at /docs to see how to update them, but basically you'd need to do a PATCH request to /streams/[id]/ with a JSON like this:

{
  "archive_urls": [
    { "url": "https://archive.org/foobar", "name": "Archive.org" },
    { "url": "https://youtube.com/foobar", "name": "YouTube" },
  ]
}

The name is optional, but I thought it could be useful for the frontend.

@cleary
Copy link
Author

cleary commented May 30, 2022

So good! Thankyou!

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

Successfully merging a pull request may close this issue.

4 participants