Skip to content

Fix event summary page crashing for dates#1142

Merged
gregorydlogan merged 2 commits intoopencast:mainfrom
Arnei:metadatasummary-crashes-for-dates
May 5, 2025
Merged

Fix event summary page crashing for dates#1142
gregorydlogan merged 2 commits intoopencast:mainfrom
Arnei:metadatasummary-crashes-for-dates

Conversation

@Arnei
Copy link
Copy Markdown
Member

@Arnei Arnei commented Mar 11, 2025

If you specify a date in the metadata when creating a new event (which you do not do with the default metadata catalog), the event summary page will crash the app. This patch should fix that.

Kind of fixes #937. Not the exact error described in the issue, which I could not reproduce, but the general problem of being unable to create a new event with date property in extended metadata.

How to test this

One way to test this is to create an extended metadata catalog in your Opencast (see https://docs.opencast.org/develop/admin/#configuration/metadata/#extended-metadata) and add a field of type "date" to it.

If you specify a date in the metadata when creating a new event
(which you do not do with the default metadata catalog),
the event summary page will crash the app. This patch should fix that.
@Arnei Arnei added the type:bug Something isn't working label Mar 11, 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-1142

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-1142

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 Mar 11, 2025

This pull request is deployed at test.admin-interface.opencast.org/1142/2025-03-19_09-35-11/ .
It might take a few minutes for it to become available.

@github-actions
Copy link
Copy Markdown
Contributor

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

@gregorydlogan gregorydlogan self-assigned this May 5, 2025
gregorydlogan added a commit to gregorydlogan/admin-interface that referenced this pull request May 5, 2025
…min-interface into main

Pull request opencast#1142

  Fix event summary page crashing for dates
@gregorydlogan gregorydlogan merged commit 9815c9e into opencast:main May 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unable to create a new event with date property in extendet metadata

2 participants