Skip to content

Treat newly created series as "synced" immediately#1623

Merged
LukasKalbertodt merged 1 commit intoelan-ev:mainfrom
owi92:faux-sync
Jan 12, 2026
Merged

Treat newly created series as "synced" immediately#1623
LukasKalbertodt merged 1 commit intoelan-ev:mainfrom
owi92:faux-sync

Conversation

@owi92
Copy link
Copy Markdown
Member

@owi92 owi92 commented Dec 17, 2025

We already know that the series has been created successfully when POST returns 201, so the series might as well be marked as "ready" even though it's not technically synced yet.
This way, users can immediately start adding videos or change metadata if they goofed up when creating the series.

We could technically send a GET request first to use the actual data that's coming from Opencast, but Tobira already knows title, (optional) description and ACL, so there's little benefit in that.

closes #1621

We already know that the series has been created
successfully when POST returns 201, so the series
might as well be marked as "ready" even though it's
not technically synced yet.
This way, users can immediately start adding videos
or change metadata if they goofed up when creating
the series.

We could technically send a GET request first to
use the actual factual data that's coming from
Opencast, but Tobira already knows title, (optional)
description and ACL, so there's little benefit in that.
@owi92 owi92 added the changelog:user User facing changes label Dec 17, 2025
@github-actions github-actions Bot temporarily deployed to test-deployment-pr1623 December 17, 2025 20:12 Destroyed
Copy link
Copy Markdown
Member

@LukasKalbertodt LukasKalbertodt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👌

@LukasKalbertodt LukasKalbertodt merged commit 46f1fb8 into elan-ev:main Jan 12, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog:user User facing changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Treat newly created series as "synced" immediately

2 participants