Skip to content

[Port main] Add citation_doi tag to head meta#4035

Merged
tdonohue merged 2 commits intoDSpace:mainfrom
alanorth:citation-doi-tag
Mar 6, 2025
Merged

[Port main] Add citation_doi tag to head meta#4035
tdonohue merged 2 commits intoDSpace:mainfrom
alanorth:citation-doi-tag

Conversation

@alanorth
Copy link
Contributor

Port of #4034 to main.

@alanorth alanorth added component: SEO Search Engine Optimization high priority labels Feb 26, 2025
@alanorth alanorth added this to the 9.0 milestone Feb 26, 2025
@alanorth alanorth self-assigned this Feb 26, 2025
@tdonohue tdonohue added the 1 APPROVAL pull request only requires a single approval to merge label Feb 26, 2025
@alanorth alanorth marked this pull request as draft March 5, 2025 11:35
@alanorth alanorth marked this pull request as ready for review March 5, 2025 13:41
alanorth added 2 commits March 5, 2025 17:01
This is used by harvesters like Altmetric and was present in DSpace
version 6 and previous.
Remove commented out this.setCitationDOITag() since it is not used
and we use camel case with this.setCitationDoiTag() now anyway.
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.

👍 Looks good. Tested via #4058 (comment) and approved via #4034

@github-project-automation github-project-automation bot moved this from 🙋 Needs Reviewers Assigned to 👍 Reviewer Approved in DSpace 9.0 Release Mar 6, 2025
@tdonohue tdonohue merged commit 987e146 into DSpace:main Mar 6, 2025
15 checks passed
@github-project-automation github-project-automation bot moved this from 👍 Reviewer Approved to ✅ Done in DSpace 9.0 Release Mar 6, 2025
@alanorth alanorth deleted the citation-doi-tag branch April 25, 2025 05:06
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 component: SEO Search Engine Optimization high priority

Projects

No open projects
Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

2 participants