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

24.12.00 harvard citations #145

Conversation

AlexanderBlanchardAC
Copy link

BEFORE APPLYING PATCH:

  1. Carry out a search and select an record. Scroll down and expand the 'Citations' section.
  2. Note that Harvard style citations are not currently an option
  3. Navigate to your account and create a list, add some items to your list.
  4. Navigate back to your list and click the title - click 'Generate Citations', note Harvard is not currently among the options.

AFTER APPLYING PATCH:

  1. Repeat the above and notice that for each record the Harvard Citation is now listed in the Citations section with the other styles.
  2. Note that Harvard citations are available in the generate citations drop down and when selected, citations for the list are now built inline with Harvard citation guidelines.

Add Haravrd Style Citations to Full Record in Citations section

feat: Alter order of citations

Alter the placement of Harvard Citations to maintain current alphabetical order
Maintain alphabetical order of dropdown
Add contributors to authors array for inclusion in citations generated from lists
After 8 authors, stop adding authors to the list and add 'et al'
Use preg_replace so Harvard citations return edn instead of edition
Update the link for the Harvard Citations style guide to use https://library-guides.ucl.ac.uk/harvard/authors
Update the style guide link for Harvard Citations to use: https://library-guides.ucl.ac.uk/harvard/a-z
Placeholder replacements failed if placeholders in text were altered during text
manipulation.
@Chloe070196
Copy link

working nicely and as expected; the one issue spotted during testing (unexpected modification of the title string) has been resolved

@AlexanderBlanchardAC AlexanderBlanchardAC merged commit 8fb012a into PTFS-Europe:24.12.00_harvard_citations Dec 3, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants