greencatsoft / scalajs-angular

AngularJS Binding for Scala.js
Apache License 2.0
252 stars 42 forks source link

Adding support for arrayKey config parameter #94

Closed thorbenheins closed 8 years ago

thorbenheins commented 8 years ago

Hi folks,

i worked around this in my project extending the original trait. Would be nice to have this as a default option. In my case spring mvc was not able to map the multipart files automatically if the the js was using file[0], file[] etc. so the arrayKey had to be overwritten with an empty string.

Maybe there even is a more elegant way to this?

Regards, Thorben

mysticfall commented 8 years ago

I left a comment in your commit, and I'd appreciate if you can review the code and make adjustments if needed.

Thanks for the contribution!

thorbenheins commented 8 years ago

Thanks for your feedback I adjusted the code accordingly. Am still trying to find my way around all the handy scala amenities.

mysticfall commented 8 years ago

Merged it and just published a new snapshot. Thanks :)