Skip to content

feat: add rest of usage metrics fields for conversation api#803

Merged
yaron2 merged 3 commits intodapr:mainfrom
sicoyle:feat-add-other-usage-fields
Feb 24, 2026
Merged

feat: add rest of usage metrics fields for conversation api#803
yaron2 merged 3 commits intodapr:mainfrom
sicoyle:feat-add-other-usage-fields

Conversation

@sicoyle
Copy link
Copy Markdown
Contributor

@sicoyle sicoyle commented Feb 23, 2026

Description

add the rest of the usage metrics fields on conversation api.

Issue reference

Please reference the issue this PR will close: #[issue number]

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

Signed-off-by: Samantha Coyle <sam@diagrid.io>
@sicoyle sicoyle requested review from a team as code owners February 23, 2026 21:26
Signed-off-by: Samantha Coyle <sam@diagrid.io>
Signed-off-by: Samantha Coyle <sam@diagrid.io>
@sicoyle
Copy link
Copy Markdown
Contributor Author

sicoyle commented Feb 23, 2026

pls ignore fossa failure that is unrelated - has occurred two times in a row now...

ssas/fossa-action@v1.3.3
/opt/hostedtoolcache/fossa/3.16.0/linux_amd64/fossa test 

Using project name: `[https://github.com/dapr/go-sdk`](https://github.com/dapr/go-sdk%60)
Using revision: `2387984d32f13baea2e7085031d9bee0a61f3705`

[ Checking build completion for 2387984d32f13baea2e7085031d9bee0a61f3705... ]
[ Waiting for build completion (revision 2387984d32f13baea2e7085031d9bee0a61f3705)... last status: StatusCreated ]
[ Waiting for build completion (revision 2387984d32f13baea2e7085031d9bee0a61f3705)... last status: StatusRunning ]
[ Waiting for build completion (revision 2387984d32f13baea2e7085031d9bee0a61f3705)... last status: StatusRunning ]
[ Waiting for build completion (revision 2387984d32f13baea2e7085031d9bee0a61f3705)... last status: StatusRunning ]
[ Waiting for issue scan completion... ]


Error:  A push-only API key was used, so issue details cannot be displayed.
  Check the webapp for issue details, or rerun this command with a full-access API key.
Error:  An issue occurred

  *** Relevant Errors ***

      Error: The scan has revealed issues. Number of issues found: 1



Error: Error: The process '/opt/hostedtoolcache/fossa/3.16.0/linux_amd64/fossa' failed with exit cod

@codecov
Copy link
Copy Markdown

codecov bot commented Feb 23, 2026

Codecov Report

❌ Patch coverage is 0% with 17 lines in your changes missing coverage. Please review.
✅ Project coverage is 54.64%. Comparing base (f60327a) to head (e64f6a8).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
client/conversation.go 0.00% 17 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #803      +/-   ##
==========================================
- Coverage   54.86%   54.64%   -0.23%     
==========================================
  Files          51       51              
  Lines        3452     3466      +14     
==========================================
  Hits         1894     1894              
- Misses       1419     1433      +14     
  Partials      139      139              

☔ 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.

@sicoyle
Copy link
Copy Markdown
Contributor Author

sicoyle commented Feb 24, 2026

ignore codecov patch failure bc it fails with 0% diff

@sicoyle
Copy link
Copy Markdown
Contributor Author

sicoyle commented Feb 24, 2026

once merged will there be another patch release @cicoyle and @JoshVanL ? I would like it for my quickstarts PR 🙏

@yaron2 yaron2 merged commit b7e3e27 into dapr:main Feb 24, 2026
21 of 24 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.

4 participants