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

Unify save requests #1472

Merged
merged 1 commit into from
Dec 5, 2024
Merged

Unify save requests #1472

merged 1 commit into from
Dec 5, 2024

Conversation

Arnei
Copy link
Member

@Arnei Arnei commented Oct 15, 2024

Fixes #485.

When the user was saving (or saving and processing), we would sent two sequential POST requests, one for metadata and one for all the rest. This unifies the requests into a single one.

Requires backend changes: opencast/opencast#6241

When the user was saving (or saving and processing),
we would sent two sequential POST requests,
one for metadata and one for all the rest.
This unifies the requests into a single one.

Requires backend changes.
@Arnei Arnei added the type:code-quality Internal improvements to the codebase label Oct 15, 2024
Copy link

This pull request is deployed at test.editor.opencast.org/1472/2024-10-15_08-53-32/ .
It might take a few minutes for it to become available.

Copy link
Contributor

@geichelberger geichelberger left a comment

Choose a reason for hiding this comment

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

LGTM

@geichelberger geichelberger merged commit ab29397 into opencast:main Dec 5, 2024
8 checks passed
geichelberger added a commit to opencast/opencast that referenced this pull request Dec 19, 2024
We have two endpoints for saving data from the editor frontend, one for
metadata and one for all the rest. With the way the editor is set up,
two seperate endpoints are wholly unnecessary, so this unifies them into
one endpoint instead.

Will break the frontend without respective changes:
opencast/opencast-editor#1472

#### Why is this aimed at legacy?

This constitutes a breaking change. This needs to be merged into
whatever version the related frontend PR is merged into.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:code-quality Internal improvements to the codebase
Projects
None yet
Development

Successfully merging this pull request may close these issues.

One request to save and start workflow
2 participants