Skip to content

[ENG-7455] easy bulk-delete long-past harvest#855

Merged
aaxelb merged 3 commits intoCenterForOpenScience:feature/share-cleanup-nograde-2025from
aaxelb:feat/delete_lotsa_old_data
Apr 23, 2025
Merged

[ENG-7455] easy bulk-delete long-past harvest#855
aaxelb merged 3 commits intoCenterForOpenScience:feature/share-cleanup-nograde-2025from
aaxelb:feat/delete_lotsa_old_data

Conversation

@aaxelb
Copy link
Contributor

@aaxelb aaxelb commented Apr 21, 2025

  • add delete_pretrove_data management command that deletes every RawDatum associated with a "pre-trove" SourceConfig (anything not ingested via the rdf-based /trove/ api)
  • update from_raw_datum FK (on IndexcardRdf-based models) with on_delete=DO_NOTHING to avoid django's emulated cascade-delete -- may error if trying to delete data still referenced

ENG-7455

@aaxelb aaxelb force-pushed the feat/delete_lotsa_old_data branch from eb20930 to 7373c65 Compare April 21, 2025 14:21
@aaxelb aaxelb force-pushed the feat/delete_lotsa_old_data branch from 7373c65 to 66d4552 Compare April 21, 2025 14:23
@coveralls
Copy link

coveralls commented Apr 21, 2025

Coverage Status

coverage: 79.141% (+0.01%) from 79.128%
when pulling 4f66df7 on aaxelb:feat/delete_lotsa_old_data
into a5d42a3 on CenterForOpenScience:feature/share-cleanup-nograde-2025.

@aaxelb aaxelb marked this pull request as ready for review April 21, 2025 14:26
@aaxelb aaxelb requested a review from felliott April 21, 2025 15:10
Copy link
Member

@felliott felliott left a comment

Choose a reason for hiding this comment

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

LGTM, but I don't have the deepest understanding of DjORM. Everything reads as sane, tho.

@aaxelb aaxelb merged commit edadaf7 into CenterForOpenScience:feature/share-cleanup-nograde-2025 Apr 23, 2025
3 checks passed
@aaxelb aaxelb deleted the feat/delete_lotsa_old_data branch April 23, 2025 18:41
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.

3 participants