Skip to content
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

Simple fix for correcting the path url for dropzone markdown images #259

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

webdevbrian
Copy link
Contributor

I don't know if this was changed or not with dropzone in their later versions from when this was first implemented, but the path is actually an object, thus we need path.url to correctly print the associated url for the file that was uploaded.

@royduin
Copy link
Contributor

royduin commented Aug 17, 2017

It's an object when you make it an object. If you just return the filepath from your controller instead of an json object or something it's working perfectly. This fix isn't needed if you ask me.

Take a look at: http://www.dropzonejs.com/#event-success, the second argument is just the response from the server.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants