Skip to content

Added streaming selection option to events#1830

Merged
tykling merged 12 commits intomasterfrom
fix_620
May 7, 2025
Merged

Added streaming selection option to events#1830
tykling merged 12 commits intomasterfrom
fix_620

Conversation

@zarya
Copy link
Copy Markdown
Collaborator

@zarya zarya commented Apr 21, 2025

Fixes #620

@zarya
Copy link
Copy Markdown
Collaborator Author

zarya commented Apr 21, 2025

After merging this to production dont forget to update the previous talks that had recording with streaming since streaming is default False!

from program.models import Event
from program.models import EventProposal

Event.objects.filter(video_recording=True).update(video_streaming=True)
Event.objects.filter(video_recording=False).update(video_streaming=False)
EventProposal.objects.filter(allow_video_recording=False).update(allow_video_streaming=False)
EventProposal.objects.filter(allow_video_recording=True).update(allow_video_streaming=True)

@tykling
Copy link
Copy Markdown
Member

tykling commented Apr 27, 2025

we should check this with the content team before merging, what does @ahf say?

@tykling
Copy link
Copy Markdown
Member

tykling commented Apr 28, 2025

After merging this to production dont forget to update the previous talks that had recording with streaming since streaming is default False!

this PR should add a datamigration for this https://docs.djangoproject.com/en/5.2/topics/migrations/#data-migrations

Comment thread src/program/models.py
Comment thread src/program/models.py
Comment thread src/program/models.py Outdated
Comment thread src/program/migrations/0106_event_video_streaming_and_more.py Outdated
pre-commit-ci Bot and others added 2 commits May 7, 2025 18:46
Comment thread src/program/migrations/0106_event_video_streaming_and_more.py Outdated
@tykling tykling merged commit 7b780c3 into master May 7, 2025
3 checks passed
@tykling tykling deleted the fix_620 branch May 7, 2025 18:52
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

Successfully merging this pull request may close these issues.

Add checkbox for livestreaming

2 participants