-
Notifications
You must be signed in to change notification settings - Fork 21
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
File upload fixes #1562
File upload fixes #1562
Conversation
Deployed to https://pr-1562.aam-digital.net/ |
5aed56d
to
700d221
Compare
Has been deployed to |
# Conflicts: # src/app/core/entity-components/entity-form/entity-form/entity-form.component.ts
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.
really, really clean code and tests. 😍
Functional testing:
- add file + cancel form--> file name still shows in field until reload
- whenever saving something other then file fields, the "apply remote changes" popup shows up now
@sleidig sorry, I forgot to update the deployment after merging the master. The issues you mentioned seem to have been fixed after merging the latest changes from the master. I couldn't reproduce them afterwards. |
SonarCloud Quality Gate failed. 0 Bugs No Coverage information |
🎉 This PR is included in version 3.14.0-master.22 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
🎉 This PR is included in version 3.14.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
The file upload has been restructured to be more in-line with other form components and to support the automatic form-update functionality.
Visible/Frontend Changes
Architectural/Backend Changes
PUT
-Requests to CouchDB are executed in sequence in order to support uploading of multiple files at the same timeEntityFormComponent
has been cleaned up to less often create new forms. Now it only creates the form one and then updates the values when needed.