search

imcvampire / vue-axios

A small wrapper for integrating axios to Vuejs
https://www.npmjs.com/package/vue-axios
MIT License
2.01k stars 182 forks source link

Confusing table in README.md #143

Closed Blitheness closed 2 years ago

Blitheness commented 2 years ago

Can this be made more explicit please? It seems unclear which versions are for which product.

image

imcvampire commented 2 years ago

Do you have any suggestions? But I think we can remove that table because v3 is released for a long time and I believe nobody uses v1 and v2 anymore.

imcvampire commented 2 years ago

https://github.com/imcvampire/vue-axios/pull/144

Blitheness commented 2 years ago

You could maybe drop v1 from that table, but there is still widespread usage of Vue 2. I contribute to some Vue 2 projects with VueAxios as a dependency.

Perhaps something like this?

VueAxios v.: 1.x 2.x 3.x
Vue 1.x ✅ ✅ ✅
Vue 2.x ✅ ✅ ✅
Vue 3.x ❌ ❌ ✅
VueAxios v.: 2.x 3.x
Vue 2.x ✅ ✅
Vue 3.x ❌ ✅
imcvampire commented 2 years ago

Doesn't vue-axios v3 support vue v2 and v3? I don't think there is anyone who still need to use vue-axios v2 if they start from scratch now