Thanks for making the add-on. It’s fairly straight forward and behaves similar to the file upload field. These are the initial issues I’ve spotted when trying to get it running :
- It uses the depreciated
$().live()
which was removed since jQuery 1.9. Should be replaced with$().on()
- see https://stackoverflow.com/questions/14354040/jquery-1-9-live-is-not-a-function - It forces ssl also on localhost (
is_ssl()
check in settings page ). Dropbox allows http when the hostname is localhost. Should also be reflected in the oauth url which defaults to https right now. Can be partially fixed / overridden withhttps://docs.gravityforms.com/gform_dropbox_ssl_compatibility/
I suppose, but the oauth url would still show “https://localhost” - Limit file extensions option has a wrong description. It says that you can define jpg,png. But the format that dropbox supports is : .jpg,.png
- There seems to be a
multiselect
option in the code-base, to allow for multiple files to be selected, but it doesn’t seem to show up in the dropbox button admin field - Embedding with ajax seems buggy, as the dropbox javascript code seems to be added after the script tries to load, and so
Dropbox is not defined
error pops up. I got around it temporarily by adding the dropbox javascript code to my wordpress header, but when I open the forms multiple times, it adds multiple dropbox buttons. Maybe move the dropbox external script to be embeddable withgravity_form_enqueue_scripts($formid, true)
instead
That’s my initial feedback