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.

Sign up for GitHub

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

Jump to bottom

[Port main] Fix redirect URLs of bitstreams with spaces in filename (new test ONLY) #3884

Merged
merged 1 commit into from
Jan 23, 2025
Merged

[Port main] Fix redirect URLs of bitstreams with spaces in filename (new test ONLY) #3884

merged 1 commit into from
Jan 23, 2025

Conversation

tdonohue
Copy link
Member

Manual port of #3876 by @msarmie to main. This port ONLY includes the new unit test, as the bug doesn't exist on main.

@tdonohue tdonohue added 1 APPROVAL pull request only requires a single approval to merge testing framework Related specifically to Unit or Integration (e2e) Tests labels Jan 22, 2025
@tdonohue tdonohue added this to the 9.0 milestone Jan 22, 2025
tdonohue
tdonohue commented Jan 23, 2025
View reviewed changes
Copy link
Member Author

@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.

👍 New test passes

@tdonohue tdonohue merged commit cb8a7cd into DSpace:main Jan 23, 2025
15 checks passed
@tdonohue tdonohue deleted the port_3876_to_main branch January 23, 2025 12:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@tdonohue tdonohue

Labels
1 APPROVAL pull request only requires a single approval to merge testing framework Related specifically to Unit or Integration (e2e) Tests
Projects
DSpace 9.0 Release
Status: ✅ Done
Milestone
9.0
Development

Successfully merging this pull request may close these issues.
2 participants
@tdonohue @msarmie