'errors' property is `any`, makes the typing useless

[jpike88]

I don't know why for such a critical property, it's just been given a type of 'any'. Is it always an array of strings? How is this acceptable?

onesignal-node-api/models/CreateNotificationSuccessResponse.ts

Line 17 in 0d5de91

'errors'?: any;

[sherwinski]

Hey @jpike88,

Thanks for the feedback here -- you're right that we can provide more information on errors per endpoint. I've made a note to investigate the different errors we can return and work this type towards something more precise.

[matheusfaggi]

In addition, returning the response with id?: string is really bad

[sherwinski]

I'm looking into how to improve the types against the different error variants that the API returns, e.g. successful notifications with invalid targets, successful notifications with no subscriptions, etc. Will follow-up with updates here.

[sherwinski]

We're making some updates to error handling that should hopefully help here. It still hasn't landed in main yet but once it gets staged, I'll follow up with some more information.