-
Notifications
You must be signed in to change notification settings - Fork 26
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
💥 Make TemporaryFileUpload.submission non-nullable #4894
💥 Make TemporaryFileUpload.submission non-nullable #4894
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #4894 +/- ##
==========================================
- Coverage 96.62% 96.62% -0.01%
==========================================
Files 757 757
Lines 25945 25942 -3
Branches 3419 3419
==========================================
- Hits 25069 25066 -3
Misses 613 613
Partials 263 263 ☔ View full report in Codecov by Sentry. |
e436584
to
171034a
Compare
|
We've provided a transition period from Open Forms 2.7, which ensures that a submission is stored. Support for legacy uploads is now removed. The celery background jobs should by now have deleted all the old unclaimed uploads, unless absurdly large retention values have been set.
Prevent upgrades going through if known constraint violations exist in the database.
171034a
to
6adb90d
Compare
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 there are some more pieces left like here:
https://github.com/open-formulieren/open-forms/blob/master/src/openforms/formio/components/vanilla.py#L387
is it not showing up in the diff for you? I've remove this check since I came across it when CTRL + F on 'legacy'. |
Part of #3283
We've provided a transition period from Open Forms 2.7, which ensures that a submission is stored. Support for legacy uploads is now removed.
The celery background jobs should by now have deleted all the old unclaimed uploads, unless absurdly large retention values have been set.
Checklist
Check off the items that are completed or not relevant.
Impact on features
Release management
I have updated the translations assets (you do NOT need to provide translations)
./bin/makemessages_js.sh
./bin/compilemessages_js.sh
Commit hygiene