clockworklabs / clockworklabs/SpacetimeDB

C#/Unity SDK callback for schema mismatch

Open
#3,160 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Rust
Stars
25.2k
Forks
1.1k
Avg merge
2d 7h
Merged PRs (30d)
46

Description

Steve from the BitCraft team raised this PR which calls the OnUnhandledReducerError when a schema mismatch caused a transaction update error in the SDK: https://github.com/clockworklabs/com.clockworklabs.spacetimedbsdk/pull/356

IMO I think we should have a specific callback for this general scenario, possibly something like OnTransactionError or something like that where we can pass these sorts of errors back to the client.

TODO

  • Determine the signature for this new callback. Consider also that we may in the future need to implement this callback in all SDKs but for now we are just focusing on the C#/Unity SDK.
  • Implement the callback
Nice to have
  • Testsuite test which demonstrates that the new callback works

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reviewing the C#/Unity SDK changes in PR #356 and the existing OnUnhandledReducerError callback. Determine the new callback signature and how schema-mismatch transaction errors reach the client. Implement the callback for the C#/Unity SDK; the nice-to-have completion criterion is a testsuite test demonstrating that it works.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp, unity
Domain
api, game-dev
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.