Skip to content
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.

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

Improved type-safety for AxiosRequestConfig #2995

Merged
merged 6 commits into from Sep 5, 2021

Conversation

carloschida
Copy link
Contributor

  • AxiosRequestConfig is now a generic type whose template corresponds to data

Signed-off-by: Carlos Chida carlos.chida@starchitecture.eu

This PR addresses a simple type-safety improvement described in #2994.

- AxiosRequestConfig is now a generic type whose template corresponds to data

Signed-off-by: Carlos Chida <carlos.chida@starchitecture.eu>
- TS tests now match the behaviour described in the PR

Signed-off-by: Carlos Chida <carlos.chida@starchitecture.eu>
@jasonsaayman jasonsaayman merged commit 4eeb3b1 into axios:master Sep 5, 2021
remcohaszing added a commit to remcohaszing/axios that referenced this pull request Oct 1, 2021
mbargiel pushed a commit to mbargiel/axios that referenced this pull request Jan 27, 2022
* Improved type-safety for AxiosRequestConfig

- AxiosRequestConfig is now a generic type whose template corresponds to data

Signed-off-by: Carlos Chida <carlos.chida@starchitecture.eu>

* Fixed tests

- TS tests now match the behaviour described in the PR

Signed-off-by: Carlos Chida <carlos.chida@starchitecture.eu>

Co-authored-by: Jay <jasonsaayman@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants