[BUG] ICheckout types are not exported about adyen-node-api-library HOT 6 CLOSED

@KadoBOT Thank you for your advices.

Regarding your previous message I was able to apply workaround:

1. import "@adyen/api-library/dist/lib/src/typings"; in specific file
2. "declaration": true in composite tsconfig.json project

from adyen-node-api-library.

Hi @splacentino

You don't import type definition files. All types are already compiled and available within the library (see here), which means you don't need to import the types to use them if you are using the library.

I just tested your code, and this works:

import { Client, CheckoutAPI } from '@adyen/api-library'

const client = new Client({ apiKey: '1234', environment: 'TEST' })
const checkout = new CheckoutAPI(client)

function getPaymentMethods(): Promise<ICheckout.PaymentMethodsResponse> {
    return checkout.paymentMethods({ merchantAccount: 'testMerchantAccount' })
}

Also, in your case, the return type is inherited, based on the type definitions of the checkout.paymentMethods. Since you manually specified the return type, you are telling the compiler that the expected return type is PaymentResponse, when the correct one is PaymentMethodsResponse, which is wrong. So in this case, you don't need to specify the return type:

from adyen-node-api-library.

Since this is not a bug, I'll close the issue. But feel free to leave any comments.

from adyen-node-api-library.

Hi @KadoBOT

I made a typo in the example, indeed.

Self types import does not look good enough. Could you try with two separate projects a and b where b is composite ts project imported in project a.

Transpilation of project b when building project a (tsc --build) will throw an error regarding types.

from adyen-node-api-library.

How the project is currently set up, it expects you to have the library installed. So it indeed it might fail for cases like yours. If you need to use the types on a different project, you would need the types to be independent, in other words, to be a package (i.e @types/@adyen/api-library).

If you feel like, you can open a feature request for that, but that would depend on more requests as is not a priority for us right now. Or, you can add the types to the @types repo yourself.

Lemme know if that helps.

from adyen-node-api-library.

@splacentino that's great! Thanks for the update.

from adyen-node-api-library.