nodejs-sdk icon indicating copy to clipboard operation
nodejs-sdk copied to clipboard

moderatorFids is missing from Channel type.

Open porenes opened this issue 1 year ago • 5 comments

Channel infos now include moderatorFids: number[] , the type needs to be updated

https://github.com/neynarxyz/nodejs-sdk/blob/bec2e51dc436e81be9ca47e5922d41d184ffa09e/src/neynar-api/v2/openapi-farcaster/models/channel.ts#L28

porenes avatar Sep 29 '24 19:09 porenes

Channel infos now include moderatorFids: number[] , the type needs to be updated

https://github.com/neynarxyz/nodejs-sdk/blob/bec2e51dc436e81be9ca47e5922d41d184ffa09e/src/neynar-api/v2/openapi-farcaster/models/channel.ts#L28

@porenes Yeah we are working on it, it should be out in a couple of hours

Shreyaschorge avatar Sep 29 '24 19:09 Shreyaschorge

You rock !

porenes avatar Sep 29 '24 19:09 porenes

@porenes are you using moderatorFids anywhere in anyway? Actually, a bug was introduced that the moderatorFids should have been snake_case but now it's camelCase. I just want to gauge whether we can make a breaking change. If so, this release will need to be put on hold. Sorry for the wait.

Shreyaschorge avatar Sep 29 '24 19:09 Shreyaschorge

It's not live yet, I am using it in my code to support the new channel model. As I am using the SDK, I used a ts-expect-error, I assume that however the api is snake_case the Typescript type will use camelCase

porenes avatar Sep 29 '24 19:09 porenes

It'll be a breaking change in api since this API returns it as a camelCase. Will discuss it with the team and let you know.

Shreyaschorge avatar Sep 29 '24 20:09 Shreyaschorge

@porenes

This should fix it. Please use snake_case. https://github.com/neynarxyz/nodejs-sdk/pull/187

Shreyaschorge avatar Oct 08 '24 09:10 Shreyaschorge