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 dspace-7_x] Fix whitespace in links due to newlines #2754

Merged
merged 2 commits into from
Jan 19, 2024

Conversation

dspace-bot
Copy link
Contributor

Port of #2721 by @alanorth to dspace-7_x.

The newlines cause whitespace to be enapsulated in the rendered link.

(cherry picked from commit 7bedf7f)
The newlines cause whitespace to be enapsulated in the rendered link.

(cherry picked from commit 7207bbb)
@dspace-bot dspace-bot added 1 APPROVAL pull request only requires a single approval to merge bug design low priority labels Jan 19, 2024
@alanorth alanorth added this to the 7.6.2 milestone Jan 19, 2024
@alanorth
Copy link
Contributor

Merging simple port of #2721. Annoying that I had to re-run the failed Node.js v16 tests three times for the item-edit test to pass (despite passing on Node.js v18).

@alanorth alanorth merged commit fed70c5 into dspace-7_x Jan 19, 2024
13 checks passed
@alanorth alanorth deleted the backport-2721-to-dspace-7_x branch January 19, 2024 07:07
4science-it pushed a commit to 4Science/dspace-angular that referenced this pull request Jan 28, 2025
[DSC-2050] fix missing queryParam in url from download metric

Approved-by: Francesco Molinaro
Approved-by: Giuseppe Digilio
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1 APPROVAL pull request only requires a single approval to merge bug design low priority
Projects
Development

Successfully merging this pull request may close these issues.

2 participants