💥 Remove already deprecated, only Axios relevant props

Author: cermakjiri

packages/@ackee/antonio-core/src/modules/request/utils/__tests__/mergeRequestConfigs.test.ts

@@ -62,14 +62,6 @@ describe('mergeRequestConfigs', () => {
         };
     });
 
-    it('assigns cancel to signal', () => {
-        const configB = {
-            cancelToken: 'cancelToken',
-        };
-
-        expect(mergeRequestConfigs(configA, configB)).toHaveProperty('signal');
-    });
-
     it('merges uri params', () => {
         const configB = {
             uriParams: { id: '1' },

packages/@ackee/antonio-core/src/modules/request/utils/mergeRequestConfigs.ts

@@ -1,6 +1,6 @@
 import Headers from 'fetch-headers';
 
-import type { RequestConfig, RequestHeaders, RequestSearchParams, FinalRequestConfig } from '../../../types';
+import type { FinalRequestConfig, RequestConfig, RequestHeaders, RequestSearchParams } from '../../../types';
 
 import type { DefaultRequestConfig } from '../config';
 
@@ -87,10 +87,5 @@ export function mergeRequestConfigs(configA: DefaultRequestConfig, configB: Requ
         result.params = mergeParams(parseParams(configA.params), parseParams(configB.params));
     }
 
-    if (configB.cancelToken) {
-        delete result.cancelToken;
-        result.signal = configB.cancelToken;
-    }
-
     return result as FinalRequestConfig;
 }

packages/@ackee/antonio-core/src/modules/response/response.ts

@@ -1,13 +1,13 @@
 import { Header } from '../../constants';
 import type { RequestConfig, RequestMethod, RequestParams, RequestResult } from '../../types';
 
-import type { ResponseInterceptors } from '../interceptors/responseInterceptors';
 import type { TAntonio } from '../core/models/Antonio';
+import type { ResponseInterceptors } from '../interceptors/responseInterceptors';
 
 import { getResponseDataType } from './responseDataTypes';
 
 import { AntonioError } from './errors';
-import { parseResponse, hasEmptyContentLength } from './utils';
+import { hasEmptyContentLength, parseResponse } from './utils';
 
 function chooseResponseDataType(config: RequestConfig, headers: Headers, requestMethod: RequestMethod) {
     if (hasEmptyContentLength(headers) || requestMethod === 'HEAD' || requestMethod === 'OPTIONS') {
@@ -80,15 +80,7 @@ export async function* processRequest<TSuccessData, TErrorData>(
         requestConfig,
     );
 
-    const result: RequestResult<TSuccessData> = {
-        request,
-        response,
-        data,
-        status: response.status,
-        statusText: response.statusText,
-        config: requestConfig,
-        headers: Object.fromEntries(response.headers.entries()),
-    };
+    const result: RequestResult<TSuccessData> = { request, response, data };
 
     return result;
 }

packages/@ackee/antonio-core/src/types.ts

@@ -146,37 +146,12 @@ export interface RequestConfig extends Omit<RequestInit, 'body' | 'headers' | 'm
     headers?: RequestHeaders;
 
     params?: RequestSearchParams;
-
-    /**
-     * @deprecated This prop is going to be removed in next major relase. Use `signal` prop instead.
-     */
-    cancelToken?: any;
 }
 
 export interface RequestResult<D> {
     request: Request;
     response: Response;
     data: D;
-
-    /**
-     * @deprecated This prop is going to be removed in next major relase. Use `response.status` instead.
-     */
-    status: number;
-
-    /**
-     * @deprecated This prop is going to be removed in next major relase. Use `response.statusText` instead.
-     */
-    statusText: string;
-
-    /**
-     * @deprecated This prop is going to be removed in next major relase. Use `response.headers` instead.
-     */
-    headers: {};
-
-    /**
-     * @deprecated This prop is going to be removed in next major relase. Depend on properties in the `request`.
-     */
-    config: {};
 }
 
 export interface FinalRequestConfig extends DefaultRequestConfig {