Skip to content

Conversation

@Anton0123
Copy link

  • feat: saga identity from attribute
  • test

@jeremydmiller
Copy link
Member

Hey, can you just reuse [SagaIdentity] here instead of an all new attribute?

@Anton0123
Copy link
Author

Anton0123 commented Dec 14, 2025

@jeremydmiller

Hey, can you just reuse [SagaIdentity] here instead of an all new attribute?

I could but I only want the property to be available when the attributes is placed on a parameter (doesn't make sense in other contexts).

Want me to move it still and should I throw an exception if the context is not a parameter or just ignore it if it's set anyways?

@jeremydmiller
Copy link
Member

@Anton0123 Nah, let's take it as is. I'll add a smidge of docs for it in a bit

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.

3 participants