search

aarondfrancis / vue-model

Model component for Vue.js
MIT License
855 stars 42 forks source link

Changed ajax dependency to vue-resource instead of jQuery #11

Closed garhbod closed 7 years ago

garhbod commented 8 years ago

Only lightly tested

thorsteneckel commented 7 years ago

Hi @garhbod - Thanks for this! Since Evan You retired vue-resource and suggested to use axios instead I think vue-model should follow this and replace $.ajax with axios instead of vue-resource. I already started to use axios for some tests which were based on your changes. Cheers for that. I will provide an axios implementation if @aarondfrancis has nothing against it and will close this PR for now.

JonasDoebertin commented 7 years ago

@thorsteneckel Any chance to get a hold of your axios implementation?

thorsteneckel commented 7 years ago

Sure! I'll create a branch and let you know. Should be done next week.

JonasDoebertin commented 7 years ago

Thanks!