Skip to content

Conversation

@cicoyle
Copy link
Collaborator

@cicoyle cicoyle commented Jul 3, 2025

support cross app workflows where app1 can host the wf and app2 will run the activity

cicoyle added 5 commits June 24, 2025 17:02
Signed-off-by: Cassandra Coyle <cassie@diagrid.io>
Signed-off-by: Cassandra Coyle <cassie@diagrid.io>
Signed-off-by: Cassandra Coyle <cassie@diagrid.io>
Signed-off-by: Cassandra Coyle <cassie@diagrid.io>
Signed-off-by: Cassandra Coyle <cassie@diagrid.io>
@cicoyle cicoyle marked this pull request as ready for review July 3, 2025 23:27
cicoyle added 2 commits July 23, 2025 13:36
Signed-off-by: Cassandra Coyle <cassie@diagrid.io>
Signed-off-by: Cassandra Coyle <cassie@diagrid.io>
Signed-off-by: Cassandra Coyle <cassie@diagrid.io>
@cicoyle cicoyle changed the title Feat cross app wf Feat cross app wf + use Java 11, not Java 8 Jul 24, 2025
@cicoyle
Copy link
Collaborator Author

cicoyle commented Jul 24, 2025

cicoyle added 2 commits July 24, 2025 12:00
Signed-off-by: Cassandra Coyle <cassie@diagrid.io>
Signed-off-by: Cassandra Coyle <cassie@diagrid.io>
Comment on lines 13 to 16
private TaskOptions(Builder builder) {
this.retryPolicy = builder.retryPolicy;
this.retryHandler = builder.retryHandler;
this.appID = builder.appID;

Choose a reason for hiding this comment

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

This private constructor looks weird, using the builder as params does not seem correct. I cannot find where we are using it

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

lmk if you are good with my last commit

cicoyle added 2 commits July 31, 2025 08:54
Signed-off-by: Cassandra Coyle <cassie@diagrid.io>
Signed-off-by: Cassandra Coyle <cassie@diagrid.io>
Copy link

@javier-aliaga javier-aliaga left a comment

Choose a reason for hiding this comment

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

LGTM

@cicoyle cicoyle merged commit 0661997 into dapr:main Jul 31, 2025
2 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.

2 participants