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

Adds almetric badge #729

Merged
merged 1 commit into from
Nov 7, 2023
Merged

Adds almetric badge #729

merged 1 commit into from
Nov 7, 2023

Conversation

justinlittman
Copy link
Contributor

closes #718

image

@justinlittman justinlittman changed the title Adds almetric badge [HOLD] Adds almetric badge Oct 24, 2023
@justinlittman justinlittman marked this pull request as ready for review October 24, 2023 18:05
@justinlittman
Copy link
Contributor Author

Note that the altmetrics badge is supposed to appear in a separate statistics section. However, since there are no statistics, I omitted the section and just put the badge on the bottom of the page.

@justinlittman justinlittman force-pushed the t718-almetric_badge branch 2 times, most recently from e0c614f to 4fdd262 Compare October 24, 2023 20:16
@justinlittman
Copy link
Contributor Author

This is before I get rid of the failure badge:
image

@justinlittman
Copy link
Contributor Author

This is after I get rid of the failure badge:
image

@justinlittman
Copy link
Contributor Author

@amyehodge Can this be un-held?

@jcoyne
Copy link
Contributor

jcoyne commented Nov 1, 2023

@justinlittman
Copy link
Contributor Author

image

@justinlittman justinlittman changed the title [HOLD] Adds almetric badge Adds almetric badge Nov 7, 2023
@amyehodge
Copy link
Collaborator

Looks good. Thanks.

@lwrubel lwrubel merged commit e9bcfcb into main Nov 7, 2023
1 check passed
@lwrubel lwrubel deleted the t718-almetric_badge branch November 7, 2023 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lwrubel lwrubel lwrubel approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Display Altmetric badge on PURL pages
4 participants
@justinlittman @jcoyne @amyehodge @lwrubel