New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Restructure/split up fetch.ts #181

Open

porcellus opened this issue Dec 28, 2022 · 0 comments

Collaborator

porcellus commented Dec 28, 2022

Current problems:

There is a lot of repeated code between interceptors
The current implementation is hard to change
fetch.ts contains multiple classes with lots of separate responsibilites

We should:

Make it clearer which classes/instances store the current state
Make it easier to change by separating responsibilities into new classes/modules
Make the possible states/state changes easier to see/follow.

The text was updated successfully, but these errors were encountered:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment