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

Entering '0' days for the submission pruning appears to have no effect #4815

Closed
sergei-maertens opened this issue Nov 8, 2024 · 4 comments · Fixed by #4819
Closed

Entering '0' days for the submission pruning appears to have no effect #4815

sergei-maertens opened this issue Nov 8, 2024 · 4 comments · Fixed by #4819

Comments

@sergei-maertens
Copy link
Member

Product versie / Product version

2.8.1

Customer reference

Taiga DH #715

Omschrijf het probleem / Describe the bug

There are global and form-specific configuration options to prune submissions (in various states, like completed/suspended/ongoing/...) after a certain number of days. The amount of days can be configured in the admin interface. A celery task (via beat) picks up the submissions that need to be pruned.

Setting the number of days to '0' seems to cause these submissions not to be pruned, while the expectation is that these would be removed on the first next run of the cleanup job.

Stappen om te reproduceren / Steps to reproduce

(unconfirmed)

  1. Create a form, set the cleanup days to '0'
  2. Start a submission and complete it
  3. Trigger the cleanup task/job
  4. Observe that the submission is not deleted

Verwacht gedrag / Expected behavior

Submission is deleted after next scheduled cleanup job run.

Screen resolution

None

Device

None

OS

None

Browser

No response

@sergei-maertens sergei-maertens added bug Something isn't working triage Issue needs to be validated. Remove this label if the issue considered valid. labels Nov 8, 2024
@sergei-maertens sergei-maertens added this to the Release 2.8.2 milestone Nov 8, 2024
@robinmolen
Copy link
Contributor

robinmolen commented Nov 8, 2024

Setting the number of days to 0 is done in the "Data removal" tab, right? When I set the value 0, I get an error message saying that the number of days should be equal or greater than 1:

Screenshot 2024-11-08 at 13 45 06

So setting the number of days to 0 shouldn't happen?

(Edit, in the global config the value 0 also isn't allowed..)

@robinmolen
Copy link
Contributor

Discussed with Sergei, we are gonna change the minimum value to 0

@joeribekker joeribekker added enhancement and removed bug Something isn't working triage Issue needs to be validated. Remove this label if the issue considered valid. labels Nov 11, 2024
robinmolen added a commit that referenced this issue Nov 11, 2024
Allowing submissions to be deleted after 0 days (i.e. on the same day)
@robinmolen robinmolen linked a pull request Nov 11, 2024 that will close this issue
9 tasks
@robinmolen
Copy link
Contributor

The minimum value for the submission removal is changed to 0.

The celery beat submission removal schedule is currently set to once a day (at 1 AM). @joeribekker @sergei-maertens do we want to set this to something like once a hour? Or even sooner, like once a minute?

@sergei-maertens
Copy link
Member Author

Once a day is fine

robinmolen added a commit that referenced this issue Nov 12, 2024
robinmolen added a commit that referenced this issue Nov 13, 2024
robinmolen added a commit that referenced this issue Nov 14, 2024
Allowing submissions to be deleted after 0 days (i.e. on the same day)
robinmolen added a commit that referenced this issue Nov 14, 2024
robinmolen added a commit that referenced this issue Nov 14, 2024
@github-project-automation github-project-automation bot moved this from Implemented to Done in Development Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants