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

Ensure exports are generated on submission delete #2132

Merged
merged 2 commits into from
Aug 16, 2021

Conversation

DavisRayM
Copy link
Contributor

Changes / Features implemented

  • Ensure exports are generated when a submission is deleted

Steps taken to verify this change does what is intended

  • Added test

Side effects of implementing this change

N/A

Before submitting this PR for review, please make sure you have:

  • Included tests
  • Updated documentation

Closes #2130

@DavisRayM DavisRayM force-pushed the 2130-exports-generate-new-on-delete branch from b9b69c2 to 640d83f Compare August 10, 2021 13:53
Usually when an instance is deleted via the `set_deleted` instance
function. The `date_modified` is also set to the time the Instance is
last saved and this is then used to determine whether an export needs to
be re-generated. This change ensures that the API performs the same
functionality as the `set_deleted` function while deleting instances
@denniswambua denniswambua self-requested a review August 13, 2021 11:19
@DavisRayM DavisRayM merged commit 1cdccd3 into master Aug 16, 2021
@DavisRayM DavisRayM deleted the 2130-exports-generate-new-on-delete branch August 16, 2021 06:53
@DavisRayM DavisRayM mentioned this pull request Aug 26, 2021
1 task
@Mstjamush
Copy link

@DavisRayM
On deletion, there's no consistency on the items deleted and the number of remaining records.
E.g on deleting one record, the total number of records drops by more than 3 record. Sometimes 3, sometimes 10 or more

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.

Update exports when records are deleted
3 participants