Skip to content

Add "Metadata saved" notification#1298

Merged
JulianKniephoff merged 7 commits intoopencast:developfrom
Arnei:metadata-saved-notification
Jun 18, 2025
Merged

Add "Metadata saved" notification#1298
JulianKniephoff merged 7 commits intoopencast:developfrom
Arnei:metadata-saved-notification

Conversation

@Arnei
Copy link
Copy Markdown
Member

@Arnei Arnei commented May 20, 2025

Adds a short notification whenever the user clicks on "Save" in the metadata tab.

Bildschirmfoto vom 2025-05-20 11-52-55

How to test this

Can be tested as usual. Try saving some metadata!

Adds a short notification whenever the user
clicks on "Save"
in the metadata tab.
@Arnei Arnei added the type:visual-clarity Improves UI readability label May 20, 2025
@github-actions
Copy link
Copy Markdown
Contributor

Use docker or podman to test this pull request locally.

Run test server using develop.opencast.org as backend:

podman run --rm -it -p 127.0.0.1:3000:3000 ghcr.io/opencast/opencast-admin-interface:pr-1298

Specify a different backend like stable.opencast.org:

podman run --rm -it -p 127.0.0.1:3000:3000 -e PROXY_TARGET=https://stable.opencast.org ghcr.io/opencast/opencast-admin-interface:pr-1298

It may take a few seconds for the interface to spin up.
It will then be available at http://127.0.0.1:3000.
For more options you can pass on to the proxy, take a look at the README.md.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented May 20, 2025

This pull request is deployed at test.admin-interface.opencast.org/1298/2025-06-18_10-29-57/ .
It might take a few minutes for it to become available.

Only display the notification after the related
web request has actually successfully completed.
Also display a warning in case the related web request fails.
Arnei added 2 commits June 4, 2025 16:54
Only display the notification after the related
web request has actually successfully completed.
Also display a warning in case the related web request fails.
None of these are necessary
@github-actions
Copy link
Copy Markdown
Contributor

This pull request has conflicts ☹
Please resolve those so we can review the pull request.
Thanks.

@github-actions
Copy link
Copy Markdown
Contributor

This pull request has conflicts ☹
Please resolve those so we can review the pull request.
Thanks.

Copy link
Copy Markdown
Member

@JulianKniephoff JulianKniephoff left a comment

Choose a reason for hiding this comment

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

LGTM now 😊

@JulianKniephoff JulianKniephoff merged commit 9ea77b3 into opencast:develop Jun 18, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:visual-clarity Improves UI readability

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants