nestjs / axios

Axios module for Nest framework (node.js) 🗂

Home Page:https://docs.nestjs.com/techniques/http-module

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Provide optional payload in delete request

teaglebuilt opened this issue · comments

Is there an existing issue that is already proposing this?

  • I have searched the existing issues

Is your feature request related to a problem? Please describe it

HTTP Wiki states that the DELETE method should have an optional payload. Why is this not the case?

Describe the solution you'd like

add data: {}` as a optional argument to delete method.

Teachability, documentation, adoption, migration strategy

No response

What is the motivation / use case for changing the behavior?

Write now, i cannot use this library to delete a resource from the api.

delete<T = any>(
url: string,
config?: AxiosRequestConfig,
): Observable<AxiosResponse<T>> {
return this.makeObservable<T>(this.instance.delete, url, config);

as you can see above, the options are from axios. Please, report it to them. npm docs axios