refactor(web): use `axios` instead of `fetch` #1530

imobachgs · 2024-08-05T14:06:38Z

Use axios instead of fetch as it implements a promise-based API which is better for our use case.
Move the logic related to interacting with the API to an API module.
Handle the ServiceStatusChanged signal properly.

To do

imobachgs added 11 commits August 5, 2024 11:19

chore(web): add axios

c1e272b

refactor(web): add an api/http module

58943ab

refactor(web): use axios to talk to the software service

b204bca

refactor(web): use axios to talk to the l10n service

63c63e5

refactor(web): use axios to fetch issues

38f7693

refactor(web): use axios to talk to the users service

50883c0

refactor(web): use axios to get the installer status

efb2b15

refactor(web): use axios to get the progress

a5002de

refactor(web): use axios to talk to the questions service

93b06cf

fix(web): handle ServiceStatusChanged events properly

6b17c4a

chore(web): drop unused imports

39958cf

imobachgs requested a review from dgdavid August 6, 2024 12:00

imobachgs marked this pull request as ready for review August 6, 2024 12:00

This comment was marked as outdated.

Sign in to view

imobachgs force-pushed the axios branch from 937d1a7 to 1a93b2d Compare August 7, 2024 08:50

chore(web): add missing type to fetchIssues

8971b7e

imobachgs force-pushed the axios branch from 1a93b2d to 8971b7e Compare August 7, 2024 09:00

imobachgs requested a review from dgdavid August 7, 2024 10:53

chore(web): add missing return type

d3c9540

dgdavid approved these changes Aug 7, 2024

View reviewed changes

imobachgs merged commit 15c0d56 into master Aug 7, 2024
2 checks passed

imobachgs deleted the axios branch August 7, 2024 13:20

imobachgs mentioned this pull request Sep 20, 2024

Update to release 10 #1622

Merged