nuxt-community / axios-module

Secure and easy axios integration for Nuxt 2
https://axios.nuxtjs.org
MIT License
1.19k stars 245 forks source link

refactor: use haxios instead of axios #535

Closed simllll closed 3 years ago

simllll commented 3 years ago

fork of nuxt axios module using haxios (based on gaxios which uses modern fetch) instead of axios

for everyone who wants to try it out, here is a published version: https://www.npmjs.com/package/@hokify/nuxt-haxios-module

it also has improved type support :)

codecov[bot] commented 3 years ago

Codecov Report

Merging #535 (5ddc7c9) into main (2796fa4) will increase coverage by 0.60%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #535      +/-   ##
==========================================
+ Coverage   88.67%   89.28%   +0.60%     
==========================================
  Files           1        1              
  Lines          53       56       +3     
  Branches       29       30       +1     
==========================================
+ Hits           47       50       +3     
  Misses          6        6              
Impacted Files Coverage Δ
lib/module.js 89.28% <100.00%> (+0.60%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 2796fa4...5ddc7c9. Read the comment docs.

seuros commented 3 years ago

I think we should have a haxios-module . It will be misleading to use haxios in the module called axios-module.

pi0 commented 3 years ago

This is nice package @simllll. I think you can create a nuxt module for it to be used ad haxios/nuxt or using options.alias['axios']='haxios' to use haxios implementation when using this module.