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

Show id and title of linked processes in metadata editor #4914

Merged

Conversation

solth
Copy link
Member

@solth solth commented Jan 18, 2022

Currently only the document type of linked processes is displayed in the metadata editor. That can become quite confusing and does not help a lot when multiple child processes of the same type are linked to one parent process, because it is not obvious which child node in the structure tree represents which child process, and thus, the order of those child processes (and potential pages in the parent process) cannot be validated/checked:

Bildschirmfoto 2022-01-18 um 13 12 21

The change in this pull request displays additional information about the linked processes, namely it's ID and process title, similar to what is displayed in when linking title records during metadata import.
This helps identifying specific child processes and structuring linked processes and pages within a parent process:

Bildschirmfoto 2022-01-18 um 13 06 53

@andre-hohmann
Copy link
Collaborator

Thanks a lot, that is a really good improvement!

Maybe it could also be regarded, that the position of an issue might need be corrected, if the order is not correct.

@solth
Copy link
Member Author

solth commented Jan 18, 2022

@andre-hohmann I think that will be possible after #4850 is merged!

@andre-hohmann
Copy link
Collaborator

Good news - thanks a lot!

@Kathrin-Huber Kathrin-Huber merged commit f4a2c1f into kitodo:master Feb 2, 2022
@solth solth deleted the linked-processes-label branch February 4, 2022 08:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants