@shreyas-chorge
We are releasing a v2 of our nodejs sdk. Looking for feedback that you might have run into with our v1.
Some of the things we already know
- API key should only be passed into only in headers
- `castHash` should be optional in `postFrameAction`
- `isPriority` should be removed from `fetchAllNotifications`, `fetchNotificationsByParentUrlForUser` and `fetchChannelNotificationsForUser`
- `karma3` should be removed from `ForYouProvider` enum (use `openrank` instead)
- `type` parameter in `fetchAllNotifications` from an enum to an array enums (OAS and SDK)
- `types` in `fetchReactionsForCast` should be `ReactionsType[]` rather than `ReactionsType`
- Remove `fetchActiveUsers`
- Accept `viewerFid` in `searchUser` as KV argument
- Accept `timeWindow` in `fetchTrendingChannels` as KV argument
- distinguish between eth and sol address naming