Skip to content
This repository has been archived by the owner on Jul 14, 2024. It is now read-only.

Full LENZ ingest #4

Merged
merged 3 commits into from
Apr 15, 2024
Merged

Full LENZ ingest #4

merged 3 commits into from
Apr 15, 2024

Conversation

richardmatthewsdev
Copy link
Collaborator

As an API user I want access to all LENZ data so I can build out demo experiences on the NZL website

Acceptance criteria

  • Ensure that security controls and steps are followed as per protocol document
  • Confirm appearance of S3 bucket that holds copy of LENZ source data
  • Consider the best way to store this source data for ongoing access
  • Consider the best way to store PDFs for ongoing access
  • NZL website demo confirms the fields needed for their search and access experiences
  • Mapping rules from source fields to API schema is confirmed with the Tech Lead before processing
  • All required data is available in the API

Copy link

Code quality score

Uh oh! The code quality got worse for this PR! Better take a look!! 🚨

Ruby file count Similarity score (flay) ABC complexity (flog) Code smells (reek) TOTALS
base 88 6.23 5.28 17.05 28.56
this branch 88 6.23 5.36 17.05 28.64
difference 0 0.0 ⚠️ 0.08 0.0 0.08

@slack-code-reviews slack-code-reviews bot requested a review from motizuki April 15, 2024 02:42
Copy link

@eoin-boost eoin-boost left a comment

Choose a reason for hiding this comment

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

@richardmatthewsdev richardmatthewsdev merged commit 0b87afc into main Apr 15, 2024
6 of 7 checks passed
@richardmatthewsdev richardmatthewsdev deleted the rm/lenz-ingest branch April 15, 2024 03:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants