getgrav / grav-plugin-email

Grav Email Plugin
http://getgrav.org
MIT License
37 stars 29 forks source link

fileupload and dropzone seems to behave strangely and automatically sending emails when deleting uploaded files from the form #136

Closed JohnMica closed 1 year ago

JohnMica commented 4 years ago

the issue seems to be related to both dropzone and how the file attachment is working.

what happens / config

image


### what happens

- fill in the form input fields with name and email
- upload a `.json` file and it errors out when uploading the file

<img width="1710" alt="Captură de ecran din 2020-07-01 la 12 01 35" src="https://user-images.githubusercontent.com/7076631/86236982-d7e5c380-bb92-11ea-9b9b-10e0a92eaa9b.png">

- delete the uploaded file and the form sends 2 emails automatically with no 'SUBMIT' button clicking

<img width="2832" alt="Captură de ecran din 2020-07-01 la 12 01 59" src="https://user-images.githubusercontent.com/7076631/86237002-e03dfe80-bb92-11ea-81b7-bb133a4289c6.png">

- trying to send the form with the new uploaded file errors since the form has already been POSTed

<img width="1975" alt="Captură de ecran din 2020-07-01 la 12 02 32" src="https://user-images.githubusercontent.com/7076631/86237019-e92ed000-bb92-11ea-9b71-1f49b0147bf6.png">

and here is clip of what is going on to simplify even more the explanation
( since github wont allow mp4 inline upload i made it a zip )

[email file upload error.mp4.zip](https://github.com/getgrav/grav-plugin-email/files/4857388/email.file.upload.error.mp4.zip)

I wonder what am i doing wrong here or if there is an issue with the way file uploads are working in this latest versions (it used to work properly in older versions of Grav v1.6.0-rc.4 - Admin v1.9.0-rc.4 )
mahagr commented 4 years ago

Which Grav and admin versions are you on now?

JohnMica commented 4 years ago

I’m testing with latest stable version downloaded yesterday from the website. Grav 1.6.26 and Admin 1.9.15. Email 3.0.9 Form 4.0.10

Regards,

John Mica

On 1 Jul 2020, at 17:36, Matias Griese notifications@github.com wrote:

 Which Grav and admin versions are you in now?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.

rhukster commented 1 year ago

Cleaning up old issues. please read: https://getgrav.org/blog/new-email-plugin