-
Notifications
You must be signed in to change notification settings - Fork 132
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
Add ability to permanently delete submissions #2437
Add ability to permanently delete submissions #2437
Conversation
6850bb5
to
3b374ac
Compare
if enable_submission_permanent_delete: | ||
instance.delete() | ||
else: | ||
error_msg = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should move this out of the loop else error_msg
will be set for each instance
3b374ac
to
4b14ce8
Compare
f7a218d
to
6880703
Compare
fed110e
to
7ef972c
Compare
7ef972c
to
0700dfc
Compare
0700dfc
to
49055c3
Compare
Signed-off-by: Kipchirchir Sigei <[email protected]>
Signed-off-by: Kipchirchir Sigei <[email protected]>
Signed-off-by: Kipchirchir Sigei <[email protected]>
Signed-off-by: Kipchirchir Sigei <[email protected]>
Signed-off-by: Kipchirchir Sigei <[email protected]>
Signed-off-by: Kipchirchir Sigei <[email protected]>
Signed-off-by: Kipchirchir Sigei <[email protected]>
49055c3
to
7734634
Compare
self.assertEqual(self.xform.instances.count(), 3) | ||
self.assertEqual(self.xform.num_of_submissions, 3) | ||
|
||
@override_settings(ENABLE_SUBMISSION_PERMANENT_DELETE=True) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's add to the documentation what setting ENABLE_SUBMISSION_PERMANENT_DELETE
is used for and its side effects
Signed-off-by: Kipchirchir Sigei <[email protected]>
7734634
to
f929305
Compare
Changes / Features implemented
ENABLE_SUBMISSION_PERMANENT_DELETE
to optionally enable/disable the functionalitySteps taken to verify this change does what is intended
Side effects of implementing this change
Before submitting this PR for review, please make sure you have:
Closes #2431