Skip to content

release: 1.33.0#156

Open
stainless-app[bot] wants to merge 3 commits into
mainfrom
release-please--branches--main--changes--next--components--node
Open

release: 1.33.0#156
stainless-app[bot] wants to merge 3 commits into
mainfrom
release-please--branches--main--changes--next--components--node

Conversation

@stainless-app
Copy link
Copy Markdown
Contributor

@stainless-app stainless-app Bot commented Apr 23, 2026

Automated Release PR

1.33.0 (2026-04-23)

Full Changelog: v1.32.0...v1.33.0

Features


This pull request is managed by Stainless's GitHub App.

The semver version number is based on included commit messages. Alternatively, you can manually set the version number in the title of this pull request.

For a better experience, it is recommended to use either rebase-merge or squash-merge when merging this pull request.

🔗 Stainless website
📚 Read the docs
🙋 Reach out for help or questions

@stainless-app stainless-app Bot requested a review from a team as a code owner April 23, 2026 13:25
@stainless-app stainless-app Bot requested review from connorlindsey and kylemcd and removed request for a team April 23, 2026 13:25
@stainless-app
Copy link
Copy Markdown
Contributor Author

stainless-app Bot commented Apr 23, 2026

🧪 Testing

To try out this version of the SDK:

npm install 'https://pkg.stainless.com/s/knock-typescript/a4a2c9668233333a46fd75dbc2f4f82af2492df5/dist.tar.gz'

Expires at: Sun, 24 May 2026 15:55:24 GMT
Updated at: Fri, 24 Apr 2026 15:55:24 GMT

* A recipient of a notification, which is either a user or an object.
*/
actor?: RecipientsAPI.RecipientReference | null;
actor?: RecipientsAPI.Recipient | null;
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Where is this coming from? In switchboard I still have this as RecipientReference (which is correct).

@connorlindsey @kylemcd anything I'm missing?

https://github.com/knocklabs/switchboard/blob/main/lib/switchboard_web/specs/workflow_recipient_runs/workflow_recipient_run.ex#L40

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@andy-knock not sure here.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the one that I was talking to chris about last week, no one could figure it out yet :/

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

iirc manual edits to the generated code will persist, so if we can't figure out the root cause, we can revert it to the right type and get this merged

@stainless-app stainless-app Bot force-pushed the release-please--branches--main--changes--next--components--node branch from eb2fb8f to 2ee5147 Compare April 23, 2026 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants