Skip to content

Add Documentation for Conversation AI SDK #1387

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
merged 10 commits into from
May 29, 2025

Conversation

siri-varma
Copy link
Contributor

Description

Add Dapr Documentation for Conversation AI SDK

Issue reference

We strive to have all PR 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: #1335

Checklist

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

  • Code compiles correctly
  • Created/updated tests
  • Extended the documentation

@siri-varma siri-varma requested review from a team as code owners May 27, 2025 04:52
@siri-varma
Copy link
Contributor Author

@dapr/maintainers-java-sdk @salaboy @artur-ciocanu @cicoyle The PR to add documentation for Conversation AI is ready for review

@siri-varma
Copy link
Contributor Author

https://docs.dapr.io/developing-applications/sdks/java/

I have already added the Jobs documentation. For both Jobs and Conversation AI (after merge) docs to show up, do I have to do anything else after this PR ?

Copy link
Contributor

@cicoyle cicoyle left a comment

Choose a reason for hiding this comment

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

@siri-varma
Copy link
Contributor Author

@cicoyle / @artur-ciocanu This PR is ready to be merged

Copy link
Contributor

@artur-ciocanu artur-ciocanu left a comment

Choose a reason for hiding this comment

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

Lovely!

@artur-ciocanu artur-ciocanu merged commit e6f7c6e into dapr:master May 29, 2025
9 checks passed
Copy link

codecov bot commented May 29, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.13%. Comparing base (d759c53) to head (3555140).
Report is 158 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #1387      +/-   ##
============================================
+ Coverage     76.91%   78.13%   +1.22%     
- Complexity     1592     1830     +238     
============================================
  Files           145      223      +78     
  Lines          4843     5663     +820     
  Branches        562      601      +39     
============================================
+ Hits           3725     4425     +700     
- Misses          821      917      +96     
- Partials        297      321      +24     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

salaboy pushed a commit to salaboy/java-sdk that referenced this pull request May 29, 2025
dapr-bot pushed a commit that referenced this pull request May 31, 2025
* adding remote endpoint request from inside activity with retry

Signed-off-by: salaboy <[email protected]>

* adding retry with Microcks payloads

Signed-off-by: salaboy <[email protected]>

* fixing review comments

Signed-off-by: salaboy <[email protected]>

* chore: New task execution task id test (#1352)

* chore: New task execution task id test

test how taskExecutionTaskId can be used for idempotency

Signed-off-by: Javier Aliaga <[email protected]>

* chore: Clean up not used files

Signed-off-by: Javier Aliaga <[email protected]>

* docs: Task execution keys

Signed-off-by: Javier Aliaga <[email protected]>

* test: Modify unit tests

Signed-off-by: Javier Aliaga <[email protected]>

* Remove new lines

Signed-off-by: artur-ciocanu <[email protected]>

---------

Signed-off-by: Javier Aliaga <[email protected]>
Signed-off-by: artur-ciocanu <[email protected]>
Co-authored-by: Cassie Coyle <[email protected]>
Co-authored-by: artur-ciocanu <[email protected]>
Signed-off-by: salaboy <[email protected]>

* Revert "chore: New task execution task id test (#1352)" (#1389)

This reverts commit 949584f.

Signed-off-by: Javier Aliaga <[email protected]>
Signed-off-by: salaboy <[email protected]>

* 1.5.5 (#1390)

Signed-off-by: Cassandra Coyle <[email protected]>
Signed-off-by: salaboy <[email protected]>

* Add Documentation for Conversation AI SDK (#1387)

Signed-off-by: salaboy <[email protected]>

* Cleanup Spring Dependencies (#1334)

* Update CONTRIBUTING.md

Signed-off-by: Siri Varma Vegiraju <[email protected]>

* Fix spring

Signed-off-by: siri-varma <[email protected]>

* Add context

Signed-off-by: siri-varma <[email protected]>

* Phase 1

Signed-off-by: siri-varma <[email protected]>

* Fix things

Signed-off-by: siri-varma <[email protected]>

* Fix things

Signed-off-by: siri-varma <[email protected]>

* Fix spring

Signed-off-by: siri-varma <[email protected]>

* Add context

Signed-off-by: siri-varma <[email protected]>

* Phase 1

Signed-off-by: siri-varma <[email protected]>

* Fix things

Signed-off-by: siri-varma <[email protected]>

* Fix things

Signed-off-by: siri-varma <[email protected]>

* move version

Signed-off-by: sirivarma <[email protected]>

* Fix pom

Signed-off-by: sirivarma <[email protected]>

* change version

Signed-off-by: sirivarma <[email protected]>

* remove unused

Signed-off-by: siri-varma <[email protected]>

* Address comments

Signed-off-by: siri-varma <[email protected]>

* Fix test

Signed-off-by: siri-varma <[email protected]>

* Fix test

Signed-off-by: siri-varma <[email protected]>

* Fix things

Signed-off-by: sirivarma <[email protected]>

---------

Signed-off-by: Siri Varma Vegiraju <[email protected]>
Signed-off-by: siri-varma <[email protected]>
Signed-off-by: sirivarma <[email protected]>
Co-authored-by: artur-ciocanu <[email protected]>
Signed-off-by: salaboy <[email protected]>

* network is needed

Signed-off-by: salaboy <[email protected]>

---------

Signed-off-by: salaboy <[email protected]>
Signed-off-by: Javier Aliaga <[email protected]>
Signed-off-by: artur-ciocanu <[email protected]>
Signed-off-by: Cassandra Coyle <[email protected]>
Signed-off-by: Siri Varma Vegiraju <[email protected]>
Signed-off-by: siri-varma <[email protected]>
Signed-off-by: sirivarma <[email protected]>
Co-authored-by: Javier Aliaga <[email protected]>
Co-authored-by: Cassie Coyle <[email protected]>
Co-authored-by: artur-ciocanu <[email protected]>
Co-authored-by: Siri Varma Vegiraju <[email protected]>
@marcduiker
Copy link
Contributor

@holopin-bot @siri-varma Thank you! Here's a digital badge as a small token of appreciation.

Copy link

holopin-bot bot commented Jun 11, 2025

Congratulations @siri-varma, the maintainer of this repository has issued you a badge! Here it is: https://holopin.io/claim/cmbs0osb2739607la8gg1xr6p

This badge can only be claimed by you, so make sure that your GitHub account is linked to your Holopin account. You can manage those preferences here: https://holopin.io/account.
Or if you're new to Holopin, you can simply sign up with GitHub, which will do the trick!

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.

Add Conversation API Java Docs
4 participants