plaid-node icon indicating copy to clipboard operation
plaid-node copied to clipboard

`RemovedTransaction.transaction_id` is optional

Open AlexLloyd0 opened this issue 1 year ago • 1 comments

The type definition for RemovedTransaction is

/**
 * A representation of a removed transaction
 * @export
 * @interface RemovedTransaction
 */
export interface RemovedTransaction {
    /**
     * The ID of the removed transaction.
     * @type {string}
     * @memberof RemovedTransaction
     */
    transaction_id?: string;
}

However the docs do not describe transaction_id as nullable:

image

This is inconsistent - do we have to handle transaction_id === undefined?

AlexLloyd0 avatar Mar 13 '24 07:03 AlexLloyd0

Thanks for reporting, and sorry for the slow response! I think that's just an oversight in the OpenAPI file. transaction_id in this context should never be undefined.

phoenixy1 avatar Mar 19 '24 20:03 phoenixy1

This has now been fixed; the fix will be available in the next client library release

phoenixy1 avatar Apr 23 '24 05:04 phoenixy1