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

Fix notes related issues in XSLT files (#1746) #1747

Merged
merged 1 commit into from
Feb 16, 2024

Conversation

anvit
Copy link
Contributor

@anvit anvit commented Jan 30, 2024

Fixes the following XSLT issues:

  • Removes "Note" subheading under the bio history section
  • Have general notes in files list the name of the type (General) instead of the type itself (generalNote)

Also split up the formatting code for elements in the collection inventory into multiple lines for readability.

@anvit anvit added Type: bug A flaw in the code that causes the software to produce an incorrect or unexpected result. work-in-progress labels Jan 30, 2024
@anvit anvit added this to the 2.8.1 milestone Jan 30, 2024
@anvit anvit self-assigned this Jan 30, 2024
@anvit anvit linked an issue Jan 30, 2024 that may be closed by this pull request
@anvit anvit changed the title Fixes notes related issues in XSLT files (#1746) Fix notes related issues in XSLT files (#1746) Feb 7, 2024
Fixes the following XSLT issues:
- Removes "Note" subheading under the bio history section
- Have general notes in files list the name of the type (General)
  instead of the type itself (generalNote)
Copy link
Contributor

@jraddaoui jraddaoui 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 to me!

</xsl:when>
<xsl:otherwise>
<xsl:value-of select="local:tagName(.)"/>
<xsl:if test="@type"> [<xsl:value-of select="local:typeLabel(.)"/>]</xsl:if>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Having a hard time checking these changes, is this the only change here?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yup, that's the only actual change here!

@anvit anvit merged commit 6ef18d0 into qa/2.x Feb 16, 2024
6 checks passed
@anvit anvit deleted the dev/issue-1746-xslt-fixes branch February 16, 2024 22:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: bug A flaw in the code that causes the software to produce an incorrect or unexpected result.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Problem: Finding aids pdfs have errors
2 participants