Skip to content

Improved type-safety for AxiosRequestConfig#2995

Merged
jasonsaayman merged 6 commits intoaxios:masterfrom
carloschida:master
Sep 5, 2021
Merged

Improved type-safety for AxiosRequestConfig#2995
jasonsaayman merged 6 commits intoaxios:masterfrom
carloschida:master

Conversation

@carloschida
Copy link
Copy Markdown
Contributor

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

Signed-off-by: Carlos Chida [email protected]

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 <[email protected]>
- TS tests now match the behaviour described in the PR

Signed-off-by: Carlos Chida <[email protected]>
@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 <[email protected]>

* Fixed tests

- TS tests now match the behaviour described in the PR

Signed-off-by: Carlos Chida <[email protected]>

Co-authored-by: Jay <[email protected]>
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.

3 participants