Skip to content

New error codes topic#4465

Merged
hhunter-ms merged 7 commits into
dapr:v1.15from
hhunter-ms:4422_ext
Dec 19, 2024
Merged

New error codes topic#4465
hhunter-ms merged 7 commits into
dapr:v1.15from
hhunter-ms:4422_ext

Conversation

@hhunter-ms

Copy link
Copy Markdown
Contributor

Description

Create new error codes topic under Developing Applications with everyone's doc contributions moved to sit underneath it.
Add an 'overview' based on @alicejgibbons presentation

Related

PR is a pt 2 of: #4422

Signed-off-by: Hannah Hunter <hannahhunter@microsoft.com>
@hhunter-ms hhunter-ms added this to the 1.15 milestone Dec 13, 2024
@hhunter-ms hhunter-ms self-assigned this Dec 13, 2024
Signed-off-by: Hannah Hunter <hannahhunter@microsoft.com>
@hhunter-ms hhunter-ms marked this pull request as ready for review December 13, 2024 20:14
@hhunter-ms hhunter-ms requested review from a team as code owners December 13, 2024 20:14
@alicejgibbons

Copy link
Copy Markdown
Contributor

Happy to review this @hhunter-ms TY for writing!

@hhunter-ms

Copy link
Copy Markdown
Contributor Author

@alicejgibbons thanks Alice! lmk if there's anything I can edit to make it clearer.

Comment thread daprdocs/content/en/developing-applications/error-codes/errors-overview.md Outdated
Comment thread daprdocs/content/en/developing-applications/error-codes/errors-overview.md Outdated
Comment thread daprdocs/content/en/developing-applications/error-codes/http-error-codes.md Outdated

```
{
"errorCode": "ERR_STATE_GET",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Need to check, but I feel this error format is the same as the one above. List this. I need to check with Alice on this, or ask Jake or watch the video again

{
  "error": "Bad Request",
  "error_msg": "{\"errorCode\":\"ERR_STATE_STORE_NOT_FOUND\",\"message\":\"state store <name> is not found\",\"details\":[{\"@type\":\"type.googleapis.com/google.rpc.ErrorInfo\",\"domain\":\"dapr.io\",\"metadata\":{\"appID\":\"nodeapp\"},\"reason\":\"DAPR_STATE_NOT_FOUND\"}]}",
  "status": 400

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@msfussell what I have, I grabbed from the video while it was paused

hhunter-ms and others added 4 commits December 19, 2024 16:44
…-overview.md

Co-authored-by: Mark Fussell <markfussell@gmail.com>
Signed-off-by: Hannah Hunter <94493363+hhunter-ms@users.noreply.github.com>
…-overview.md

Co-authored-by: Mark Fussell <markfussell@gmail.com>
Signed-off-by: Hannah Hunter <94493363+hhunter-ms@users.noreply.github.com>
…rror-codes.md

Co-authored-by: Mark Fussell <markfussell@gmail.com>
Signed-off-by: Hannah Hunter <94493363+hhunter-ms@users.noreply.github.com>

@msfussell msfussell left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@hhunter-ms hhunter-ms merged commit 5dd13ad into dapr:v1.15 Dec 19, 2024
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.

3 participants