-
Notifications
You must be signed in to change notification settings - Fork 23
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
Split presenters at ";" #1104
base: main
Are you sure you want to change the base?
Split presenters at ";" #1104
Conversation
It was brought to my attention that this was a feature in the previous admin ui, so this patch adds it back in. When adding multiple presenters to an event in the metadata tab, you can now add multiple presenters at the same time by delimiting them with a ";". For example, writing "Pacman; Duck Hunt" would result in both "Pacman" and "Duck Hunt" being added as presenters. Will also work for other metadata fields that work the same way, e.g. contributers.
Use Run test server using develop.opencast.org as backend:
Specify a different backend like stable.opencast.org:
It may take a few seconds for the interface to spin up. |
This pull request is deployed at test.admin-interface.opencast.org/1104/2025-02-07_09-13-13/ . |
This patch enable splitting value by an |
The delimiter configuration value will be parsed by Opencast backend and provided by metadata endpoints liek [
{
"flavor": "dublincore\/episode",
"title": "EVENTS.EVENTS.DETAILS.CATALOG.EPISODE",
"fields": [
{
"readOnly": false,
"id": "title",
"label": "EVENTS.EVENTS.DETAILS.METADATA.TITLE",
"type": "text",
"value": "",
"required": true
},
{
"delimiter": ";",
"readOnly": false,
"id": "contributor",
"label": "EVENTS.EVENTS.DETAILS.METADATA.CONTRIBUTORS",
"type": "mixed_text",
"value": [],
"required": false
}
]
}
] |
I cannot reproduce that. As far as I can tell this really only affects metadata fields of the Bildschirmaufzeichnung.vom.2025-02-07.09-16-51.webm
I was not aware that this was configurable, thanks for telling me! I will make sure the new admin ui respects the configuration. |
Apparently the delimiter for mixed_text fields is not supposed to be hardcoded as ";", but is actually specified in the metadata catalog configuration. As such, this patch fixes the previous commit in that the delimiter is now respected (and in case no delimiter is specified, no splitting is happening).
300de36
to
80aac56
Compare
It was brought to my attention that this was a feature in the previous admin ui, so this patch adds it back in.
When adding multiple presenters to an event in the metadata tab, you can now add multiple presenters at the same time by delimiting them with a ";". For example, writing "Pacman; Duck Hunt" would result in both "Pacman" and "Duck Hunt" being added as presenters. Will also work for other metadata fields that work the same way, e.g. contributers.
EDIT: With the lastest commit, the delimiter is not ";", but instead specified in the metadata catalog configuration (where all metadata catalog fields are specified). If no delimiter is specified, no splitting will take place.
How to test this
Can be tested as usual.