-
Notifications
You must be signed in to change notification settings - Fork 37
Upgrade auto-save for surveys and forms #280
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
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #280 +/- ##
===========================================
- Coverage 96.02% 93.46% -2.56%
===========================================
Files 97 125 +28
Lines 2440 3521 +1081
===========================================
+ Hits 2343 3291 +948
- Misses 97 230 +133 ☔ View full report in Codecov by Sentry. |
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.
@ElviaBth this PR needs more things
On one hand, it is introducing changes related to the integram chat, this should not be here (see some comments).
On the other hand, this feature is about saving temporary data on forms using the localstorage browser capabilities. This is done with the script "autosave.js" (which uses https://github.com/appleple/form-storage in a npm package).
You need to:
- Upgrade the formstorage package (if possible) in package.json
- Adapt the
autosave.jsfile so it removes jQuery dependencies - Uncomment the import in the
decidim_application.jsfile
Thanks!
🎩 What? Why?
Auto-save for surveys and forms.
📌 Related Issues
#/270 Upgrade 0.28 pending task 2.
📷 Screenshots

