-
Notifications
You must be signed in to change notification settings - Fork 130
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
Outline delete
route for H2.0 flask app
#4644
Comments
Should "deleting" automatically run a clear? Should a "clear" route exist to clear a harvest source? What would a clear do? What would either a delete or clear do to the harvest jobs, records, and errors? |
|
we want to hard delete the records with a cascade to start. we would also want to make sure solr is updated to reflect the deletions. |
@Jin-Sun-tts is working on this in #4654 |
define the relationship with |
need to add ckan api call to delete all children datasets |
open a new ticket to Implement dataset deletion on organization/harvest-source removal #4691. |
User Story
In order to remove harvest sources to our Harvest DB, datagovteam wants to create a new Flask app to handle basic delete of Harvest Source configs.
Links:
Acceptance Criteria
[ACs should be clearly demoable/verifiable whenever possible. Try specifying them using BDD.]
GIVEN a definition of what
delete
means in this contextTHEN I can proceed with the creation of the routes below
GIVEN I call the route
{url}/harvests/delete
with a corresponding IDTHEN the above is implemented
Background
Continuing the work in #4634, what deleting a record needs to be defined. We have a number of possible options available: true delete from the db, a
deleted
column/flag, moved to an archive table, etc. This ticket is intended to think more deeply about what option best suits our use case, get buy in from the team, and finally implement the solution.Security Considerations (required)
[Any security concerns that might be implicated in the change. "None" is OK, just be explicit here!]
Sketch
delete
The text was updated successfully, but these errors were encountered: