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

FIND-81:Add record attributes from CIIM to BE #15

Open
wants to merge 35 commits into
base: main
Choose a base branch
from

Conversation

TNA-Allan
Copy link
Contributor

@TNA-Allan TNA-Allan commented Dec 10, 2024

Pull Request

Ticket URL: https://national-archives.atlassian.net/browse/FIND-81

About these changes

As part of the transfer from https://github.com/nationalarchives/ds-wagtail to use Rosetta API

This PR is BE focused to getting the record attributes from API response available to use in the record details template app/templates/records/record_detail.html. The attributes return raw data. There are separate tickets to address those attributes returning html in the data - the mark_safe/safe filter is differed until then.

  • Adds records attributes from CIIM api response
  • Exposes the record template variable to use in the record details template
  • Refactors, adds tests - some improvements
  • Update docs

How to check these changes

This PR is BE focused, it can be tested as follows-
Ref to Catalogue Data Elements for record attributes and iaid examples
http://localhost:65533/catalogue/id/[iaid-value]/
add {{ record.[attribute-name] }} to app/templates/records/record_detail.html

Before assigning to reviewer, please make sure you have

  • Checked things thoroughly before handing over to reviewer
  • Checked PR title starts with ticket number as per project conventions to help us keep track of changes
  • Ensured that PR includes only commits relevant to the ticket
  • Waited for all CI jobs to pass before requesting a review
  • Added/updated tests and documentation where relevant

@TNA-Allan TNA-Allan changed the title FIND-81:Add CIIM record attributes to BE FIND-81:Add record attributes from CIIM to BE Dec 10, 2024
@TNA-Allan TNA-Allan marked this pull request as ready for review December 11, 2024 12:13
@TNA-Allan TNA-Allan requested a review from Firdaussi December 11, 2024 12:25
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.

1 participant