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

do not display history in charter previews #104

Closed
holfordm opened this issue Oct 15, 2023 · 2 comments
Closed

do not display history in charter previews #104

holfordm opened this issue Oct 15, 2023 · 2 comments
Assignees

Comments

@holfordm
Copy link
Collaborator

(examples where it displays: https://charters-qa.bodleian.ox.ac.uk/?f%5Bms_collection_s%5D%5B%5D=Suffolk&f%5Btype%5D%5B%5D=charter&page=16&per_page=200) -- apparently because of the format of the history selection with an acquistion element and no provenance elements (which can be changed in the data, but might be useful to change whatever is causing the display to avoid future issues)

holfordm added a commit that referenced this issue Oct 15, 2023
hopefully deals with #104
@Slange-Mhath
Copy link
Collaborator

Kind of, the problem here is again that Phyiscal Description is not in the data (because it probably does not have one). Regex is super powerful so my current quick fix solution would be that I adjust the regex to check if Physical Description is somewhere in the data, and if so use that as ending for the preview text (not including) or if its not in the data use History as ending (not including). While that will probably work it can cause side effects. Another "problem" with regex is that it can potentially slow things down a bit - my long term solution would be that I open another issue discussing how we could use another preview field in the data which we'd need to create in the data once and can then just display in blacklight. For now though, I just adjust the regex.

@Slange-Mhath
Copy link
Collaborator

Slange-Mhath commented Oct 16, 2023

Thats the short term fix: https://gitlab.bodleian.ox.ac.uk/TEI-consolidation/charters-catalogue-search/-/commit/a48188f3d4206ba60b68533f831e8737906ba2fa and it should work as soon as its deployed on qa and prd

And the long term fix I'd propose for sometime soon: #106

@holfordm holfordm closed this as completed May 8, 2024
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

No branches or pull requests

2 participants