Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Change Harvest Record Errors DB Relationship #5072

Open
6 tasks
btylerburton opened this issue Feb 6, 2025 · 2 comments
Open
6 tasks

Change Harvest Record Errors DB Relationship #5072

btylerburton opened this issue Feb 6, 2025 · 2 comments
Assignees
Labels
H2.0/Harvest-DB Postgres and related DB tickets

Comments

@btylerburton
Copy link
Contributor

User Story

In order to retain errors about the harvest job once a record has been deleted, datagovteam wants to change the relationship for a Harvest Record Error from a Harvest Record to a Harvest Job.

Acceptance Criteria

[ACs should be clearly demoable/verifiable whenever possible. Try specifying them using BDD.]

  • GIVEN a harvest record error has been created
    WHEN the corresponding record has been deleted
    THEN I will still see the error

  • GIVEN a harvest record error has been created
    WHEN the corresponding job has been deleted
    THEN I will no longer see the error

Background

[Any helpful contextual notes or links to artifacts/evidence, if needed]

Security Considerations (required)

[Any security concerns that might be implicated in the change. "None" is OK, just be explicit here!]

Sketch

  • Modify harvest_job model to support job_errors and record_errors
  • Create a new cascading delete relationship with record_errors
  • Add new foreign-key relationship in HarvestRecordError to harvest_job.id
  • Ensure the tests are updated
@btylerburton btylerburton added the H2.0/Harvest-DB Postgres and related DB tickets label Feb 6, 2025
@Bagesary
Copy link

Bagesary commented Feb 6, 2025

This should be implemented before the MVP release

@Bagesary Bagesary moved this to 📥 Queue in data.gov team board Feb 6, 2025
@cmhedrickREI cmhedrickREI self-assigned this Feb 7, 2025
@cmhedrickREI cmhedrickREI moved this from 📥 Queue to 🏗 In Progress [8] in data.gov team board Feb 7, 2025
@Bagesary
Copy link

Bagesary commented Feb 7, 2025

This will need a data migration. No blockers as of now

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
H2.0/Harvest-DB Postgres and related DB tickets
Projects
Status: 🏗 In Progress [8]
Development

No branches or pull requests

3 participants