ie8 and ie9 file upload fixes #158

Merged
merged 3 commits into from Feb 15, 2013

2 participants

@markotibold
  • Upgrade to latest stable Fine Uploader
  • Fix an ie8 and ie9 bug where responses to file uploads must be of content type 'text/plain' (https://github.com/valums/file-uploader/#internet-explorer-limitations)
  • Added a customFields option through which we can pass the csrf token to the form uploader because. (fine uploader does not use xhr with ie8 and ie9 for uploads). Had to modify the code of fine uploader slightly, but a PR to their repo will be issued shortly as well.
@dbunskoek
Ride The Pony member

Hi Marko, this is great :)

I saw that debug is set to true, should that be removed, or is it necessary?

EDIT: whoops, sorry, only looked at the first commit :)

@dbunskoek dbunskoek merged commit 61c54f6 into ridethepony:dev Feb 15, 2013

1 check passed

Details default The Travis build passed
@markotibold

issued a PR at valums file uploader here: https://github.com/valums/file-uploader/pull/674

@markotibold

Dennis, debug used to be set to 'true' it's false now. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment