bcgov / bcgov/entity

Create Business UI: change axios to built-in fetch

Open
#2,707 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
23
Forks
62
Avg merge
24m
Merged PRs (30d)
1

Description

## Description:

Per Thor:
> whoa, in a diff project I just dropped axios for the built-in fetch (good bye so many cors incantations) and vue can polyfill for old decrepit browsers - import 'whatwg-fetch'

Re: https://www.npmjs.com/package/whatwg-fetch

I estimate 2 days due to needing to refactor the unit tests (some of which mock/stub axios).

Once we have an idea how big a change this is, we may want to make this same change to Coops UI ... need to create a task for that.

Acceptance for a Task:
- [ ] Requires deployments
- [ ] Add/ maintain selectors for QA purposes
- [ ] Test coverage acceptable
- [ ] Linters passed
- [ ] Peer Reviewed
- [ ] PR Accepted
- [ ] Production burn in completed

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.