Skip to content

Add data to EventProperties #1067

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

Conversation

fjtirado
Copy link
Collaborator

@fjtirado fjtirado commented Jan 14, 2025

Please specify parts of this PR update:

  • Specification
  • Schema
  • Examples
  • Extensions
  • Use Cases
  • Community
  • CTK
  • Other

Discussion or Issue link:

What this PR does:

Additional information:

Signed-off-by: Francisco Javier Tirado Sarti <ftirados@redhat.com>
@fjtirado fjtirado force-pushed the Add_data_to_event_properties branch from eaf7322 to a96f613 Compare January 14, 2025 16:10
@fjtirado fjtirado force-pushed the Add_data_to_event_properties branch from 3da6cad to 7cbedcd Compare January 14, 2025 17:08
@fjtirado fjtirado force-pushed the Add_data_to_event_properties branch from 7cbedcd to 37f7cf3 Compare January 14, 2025 17:18
@fjtirado fjtirado force-pushed the Add_data_to_event_properties branch from 96e0fc5 to 241c109 Compare January 14, 2025 18:03
Copy link
Member

@cdavernas cdavernas left a comment

Choose a reason for hiding this comment

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

LGTM! Cheers!

@fjtirado fjtirado force-pushed the Add_data_to_event_properties branch 5 times, most recently from 0f8f5bd to 11c1521 Compare January 14, 2025 18:34
Copy link
Collaborator

@matthias-pichler matthias-pichler left a comment

Choose a reason for hiding this comment

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

Not putting any restrictions on data would also work

Co-authored-by: Charles d'Avernas <charles.davernas@neuroglia.io>
Signed-off-by: Francisco Javier Tirado Sarti <ftirados@redhat.com>
@fjtirado fjtirado force-pushed the Add_data_to_event_properties branch from 11c1521 to 91c6cf4 Compare January 14, 2025 18:39
@fjtirado
Copy link
Collaborator Author

fjtirado commented Jan 14, 2025

Not putting any restrictions on data would also work

I was trying different approaches. I think

        anyOf:
          - $ref: '#/$defs/runtimeExpression'
          - {}

emphasize the possiibilty of using an expression.
Basically {} is similar to former and not longer supporter type:any ( I did not know that)

@cdavernas cdavernas merged commit a0e15df into serverlessworkflow:main Jan 14, 2025
3 checks passed
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