Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Update state management Readme with multi app run #904

Merged
merged 12 commits into from
Sep 28, 2023
Merged

Update state management Readme with multi app run #904

merged 12 commits into from
Sep 28, 2023

Conversation

rabollin
Copy link
Contributor

@rabollin rabollin commented Sep 10, 2023 •
edited
Loading

Description

This PR is to add a sample example of running Multi app run to statestore quickstart

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • The quickstart code compiles correctly
  • You've tested new builds of the quickstart if you changed quickstart code
  • You've updated the quickstart's README if necessary
  • If you have changed the steps for a quickstart be sure that you have updated the automated validation accordingly. All of our quickstarts have annotations that allow them to be executed automatically as code. For more information see mechanical-markdown. For user guide with examples see Examples.

This PR is opened to address the issue: #905

@rabollin rabollin changed the title DRAFT PR - Statestore quickstart DRAFT PR - Updating state management Readme with multi app run Sep 10, 2023
@hhunter-ms hhunter-ms mentioned this pull request Sep 21, 2023
Merged
3 tasks
@rabollin rabollin changed the title DRAFT PR - Updating state management Readme with multi app run Update state management Readme with multi app run Sep 21, 2023
@paulyuk paulyuk added the P1 label Sep 22, 2023
@rabollin rabollin changed the base branch from master to release-1.12 September 22, 2023 13:26
paulyuk
paulyuk requested changes Sep 23, 2023
View reviewed changes
Copy link
Contributor

@paulyuk paulyuk left a comment

Choose a reason for hiding this comment

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

Please see comments

state_management/csharp/http/README.md Show resolved Hide resolved
state_management/go/http/README.md Show resolved Hide resolved
state_management/go/http/order-processor/order_processor_example.exe Outdated Show resolved Hide resolved
state_management/csharp/sdk/order-processor/Program.cs Outdated Show resolved Hide resolved
state_management/go/sdk/order-processor/app.go Outdated Show resolved Hide resolved
...nt/java/sdk/order-processor/src/main/java/com/service/OrderProcessingServiceApplication.java Outdated Show resolved Hide resolved
state_management/javascript/http/order-processor/index.js Outdated Show resolved Hide resolved
state_management/javascript/sdk/order-processor/index.js Outdated Show resolved Hide resolved
state_management/python/http/order-processor/app.py Outdated Show resolved Hide resolved
state_management/python/sdk/order-processor/app.py Outdated Show resolved Hide resolved
@paulyuk paulyuk added this to the 1.12 milestone Sep 23, 2023
@rabollin
pr comments
05bf320 
Signed-off-by: rabollin <[email protected]>
@rabollin
pr comments
bc917d2 
Signed-off-by: rabollin <[email protected]>
@rabollin
sleep value increase
dcfb6d3 
Signed-off-by: rabollin <[email protected]>
@rabollin
sleep seconds increase
ab6b09e 
Signed-off-by: rabollin <[email protected]>
@rabollin
Result correction
972f197 
Signed-off-by: rabollin <[email protected]>
@rabollin
Result correction
80e7d45 
Signed-off-by: rabollin <[email protected]>
@rabollin
java sdk dapr run output format fixing
1086072 
Signed-off-by: rabollin <[email protected]>
@paulyuk
Update README.md
df74c45 
Fixing typo.  should be singular

Signed-off-by: Paul Yuknewicz <[email protected]>
@paulyuk
Update README.md
f0261c7 
Signed-off-by: Paul Yuknewicz <[email protected]>
paulyuk
paulyuk approved these changes Sep 27, 2023
View reviewed changes
Copy link
Contributor

@paulyuk paulyuk left a comment

Choose a reason for hiding this comment

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

Approved with my tweaks

state_management/java/sdk/README.md Outdated Show resolved Hide resolved
state_management/csharp/sdk/order-processor/Program.cs Outdated Show resolved Hide resolved
state_management/go/sdk/README.md Outdated Show resolved Hide resolved
state_management/javascript/sdk/README.md Outdated Show resolved Hide resolved
@paulyuk paulyuk merged commit da2f79e into dapr:release-1.12 Sep 28, 2023
7 checks passed
@paulyuk paulyuk mentioned this pull request Jul 29, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@paulyuk paulyuk paulyuk approved these changes

Assignees

@rabollin rabollin

Labels
P1
Projects
None yet
Milestone
1.12
Development

Successfully merging this pull request may close these issues.
2 participants
@rabollin @paulyuk