Skip to content

Conversation

@SteveGibsonCL
Copy link
Contributor

Description of Changes

  • Changed C# reducer signature to include only ReducerEvent and reducer-specific arguments. Generated events also now have delegates for better UX
  • Added Timestamp and Identity properties to ReducerEvent
  • Added None to ReducerType enum to avoid potential corruption from uninitialized ReducerEvent instances
  • Change C# reducer invocation method to return whether or not reducer has subscribers. This allows us to have an event for handling errors from reducers without handlers (see SDK PR #31)
  • Moved all ReducerEvent and ReducerArgs code into a single file for better UX

API

  • This is a breaking change to the module API
  • This is a breaking change to the ClientAPI

If the API is breaking, please state below what will break

…urn wether or not reducer has subscribers; Moved all ReducerEvent and ReducerArgs code into a single file
@SteveGibsonCL SteveGibsonCL mentioned this pull request Jun 22, 2023
2 tasks
Copy link
Collaborator

@jdetter jdetter left a comment

Choose a reason for hiding this comment

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

Tested against d9a4e4b161c8956ae6f6ace5c5286843bd443a11, works for me 👍

@SteveGibsonCL SteveGibsonCL merged commit 5eee895 into master Jun 26, 2023
cloutiertyler pushed a commit that referenced this pull request Aug 1, 2023
…urn wether or not reducer has subscribers; Moved all ReducerEvent and ReducerArgs code into a single file (#10)

Co-authored-by: Steve <steve@codefics.com>
cloutiertyler pushed a commit that referenced this pull request Aug 1, 2023
…urn wether or not reducer has subscribers; Moved all ReducerEvent and ReducerArgs code into a single file (#10)

Co-authored-by: Steve <steve@codefics.com>
@cloutiertyler cloutiertyler deleted the change-csharp-reducer-signature branch August 1, 2023 21:54
bfops pushed a commit that referenced this pull request Jul 16, 2025
bfops pushed a commit that referenced this pull request Jul 16, 2025
The attribute no longer exist, we now either infer the name automatically or specify it within the `table(scheduled(at = ...))` attribute instead.
bfops pushed a commit that referenced this pull request Jul 17, 2025
Co-authored-by: John Detter <no-reply@boppygames.gg>
bfops pushed a commit that referenced this pull request Aug 7, 2025
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.

5 participants