quickstarts icon indicating copy to clipboard operation
quickstarts copied to clipboard

Add quickstart for Conversation AI java sdk

Open siri-varma opened this issue 7 months ago • 6 comments

Description

Add quickstart for Conversation AI sdk

Issue reference

We strive to have all PRs being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close: #1155

Checklist

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

  • [X] The quickstart code compiles correctly
  • [X] You've tested new builds of the quickstart if you changed quickstart code
  • [X] You've updated the quickstart's README if necessary
  • [X] 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.

siri-varma avatar May 27 '25 19:05 siri-varma

@salaboy @cicoyle @artur-ciocanu Can you folks take a look at this PR please ?

siri-varma avatar May 27 '25 19:05 siri-varma

@cicoyle Could you please approve the CI one more time ? I validated it locally. Should succeed now.

siri-varma avatar May 27 '25 21:05 siri-varma

@siri-varma same.. from a pure Java standpoint this PR LGTM, ready to be merged!

salaboy avatar May 28 '25 06:05 salaboy

image

Validated locally

siri-varma avatar May 28 '25 18:05 siri-varma

@alicejgibbons small request, could you please trigger the build for this PR ? Want to make sure it passes so that we will not have any surprises at the end

siri-varma avatar Jun 02 '25 17:06 siri-varma

Im honestly fine merging this and then once I cut the official java release we can come back and update the versions everywhere - that way this PR doesnt get blocked

cicoyle avatar Jun 24 '25 21:06 cicoyle

@alicejgibbons Do you think we can go ahead and merge this PR now ? I will update the sdks to the latest version once 1.15 is out

siri-varma avatar Jul 10 '25 18:07 siri-varma

@siri-varma we want to align this PR with the 1.16 conversation changes so will pull those in when we get a 1.16.0 RC 2

alicejgibbons avatar Jul 28 '25 11:07 alicejgibbons

Need to add tool calling to this quickstart for 1.16 as described here: https://github.com/dapr/quickstarts/issues/1213 and retarget 1.16 branch

alicejgibbons avatar Aug 07 '25 18:08 alicejgibbons

Same with this one - if you do the retargeting to 1.16 branch then im happy to add the tool calling bits after your merge!

alicejgibbons avatar Aug 15 '25 13:08 alicejgibbons

@alicejgibbons Its done

siri-varma avatar Aug 18 '25 04:08 siri-varma

Merging this one since the csharp quickstart is failing due to #1212

alicejgibbons avatar Aug 20 '25 13:08 alicejgibbons