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

[Port main] Remove Bitstreams on edit-item-page doesn't work #3127

Conversation

alexandrevryghem
Copy link
Member

@alexandrevryghem alexandrevryghem commented Jun 17, 2024

Fixes #3116
Manual port of #3121 tomain.

Also contains a fix for the bitstream routing module sometimes complaining about Error: NG04014: Invalid configuration of route 'bitstreams/handle/:prefix/:suffix/:filename'. One of the following must be provided: component, loadComponent, redirectTo, children or loadChildren

…& removed method calls retuning observables in edit collection's content source tab
@alexandrevryghem alexandrevryghem self-assigned this Jun 17, 2024
@alexandrevryghem alexandrevryghem added bug high priority component: Item (Archived) Item display or editing labels Jun 17, 2024
@alexandrevryghem alexandrevryghem added this to the 8.0 milestone Jun 17, 2024
@tdonohue tdonohue self-requested a review June 17, 2024 17:33
Copy link
Member

@tdonohue tdonohue left a comment

Choose a reason for hiding this comment

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

👍 Thanks @alexandrevryghem ! Tested this PR on top of latest main and verified this fixes the bug. The code looks good & matches with what is in #3121

@tdonohue tdonohue merged commit cd3c4f2 into DSpace:main Jun 18, 2024
13 checks passed
@alexandrevryghem alexandrevryghem deleted the w2p-113560_edit-item-add-relationships-one-by-one_contribute-main branch June 18, 2024 17:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug component: Item (Archived) Item display or editing high priority
Projects
No open projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

Remove Bitstreams on edit-item-page doesn't work
2 participants