Alexandre Vryghem
Alexandre Vryghem
I thought about 2 additional minor improvements that I can add to this PR if there are no objections: - We could maybe hide the `Reset filters` button when `inPlaceSearch`...
@tdonohue: I retested it locally after merging the latest main and fixed a remaining spacing issue on the home page and also already implemented [these suggestions](https://github.com/DSpace/dspace-angular/pull/2844#issuecomment-1972187510)
@tdonohue: Sry I don't know what happend 😬 I cherry-picked the changes back on the latest version of `main`
@tdonohue: I merged the latest `main` branch and the error was automatically fixed, so this was probably because of a recent change on `main`
@kanasznagyzoltan: This is another issue see #2953 but I already created a fix for this and was planning to contribute it to this PR later today
@f-rapp: With PR #2888 the current breadcrumb is completed with the comcol structure, would this be sufficient?
@tdonohue: I retested this myself today and for me the redirect now works correctly on sandbox, I think there might have been a confusion because there are 2 characters that...
@tdonohue: I just retested it now too, but for me it still returns 302 for me so that's weird 😅 ```sh → curl --head https://sandbox.dspace.org/bitstream/handle/10673/1131/test_pdf.pdf HTTP/2 302 date: Mon, 20...
PR #3060 refactored the way relationships are added on items and also fixed this bug. So I'm closing this PR since it will conflict with that PR and this PR...
I already fixed this on 8.0 in #2732, so I'm assigning myself in order to create a backport for the `dspace-7_x` branch