Skip to content

Updates the TypesScript SDK to use the new EventContext variants #136

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 29 commits into from
Feb 8, 2025

Conversation

cloutiertyler
Copy link
Contributor

@cloutiertyler cloutiertyler commented Feb 8, 2025

Description of Changes

Diff is largely code gen changes based on the changes as a result of: clockworklabs/SpacetimeDB#2224 The actual material changes ended up being relatively small thanks to TypeScript's crazy powerful type system.

That PR also has a more full description of the changes made and their impact.

This one should merge BEFORE #135

API

  • This is an API breaking change to the SDK

See the description of clockworklabs/SpacetimeDB#2224 which lays out the breaking changes.

Requires SpacetimeDB PRs

clockworklabs/SpacetimeDB#2224

Copy link

changeset-bot bot commented Feb 8, 2025

⚠️ No Changeset found

Latest commit: 3e921af

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@cloutiertyler cloutiertyler changed the title Tyler/event context update Updates the TypesScript SDK to use the new EventContext variants Feb 8, 2025
@cloutiertyler
Copy link
Contributor Author

Phoebe actually already approved this in #132 but I didn't realize that wasn't into the main branch.

Copy link

pkg-pr-new bot commented Feb 8, 2025

Open in Stackblitz

npm i https://pkg.pr.new/@clockworklabs/spacetimedb-sdk@136

commit: 3e921af

@cloutiertyler
Copy link
Contributor Author

Given that @gefjon previously approved the earlier version of this one, I am merging it.

@cloutiertyler cloutiertyler merged commit 1548f61 into main Feb 8, 2025
7 of 8 checks passed
@cloutiertyler cloutiertyler deleted the tyler/event-context-update branch February 8, 2025 05:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant