Skip to content

[DIT-12111] Improve error messages, particularly for errors from the Ditto API#142

Merged
joustrich merged 1 commit into
masterfrom
joustrich/DIT-12111-error-messages
Jul 20, 2026
Merged

[DIT-12111] Improve error messages, particularly for errors from the Ditto API#142
joustrich merged 1 commit into
masterfrom
joustrich/DIT-12111-error-messages

Conversation

@joustrich

Copy link
Copy Markdown
Contributor

Overview

This PR improves error messages, particularly for errors from the Ditto API.

Context

Previously any errors from the Ditto API would have logged out a generic error message, now it will instead log whatever error message is returned from the API.

Screenshots

Before:
ditto-cli-project-error-before

After:
ditto-cli-project-error-after

Test Plan

Testing successfully completed in dev/local via:

  • Add an invalid project ID to your ditto/config.yml such as id: "invalid-project-id"
  • Run the pull command (or you can use yarn sync if running directly from this repo)
  • You should now see an error message that mentions you have an invalid project ID, like you can see in the "After" screenshot above. Previously this would have been the generic error you see in the "Before" screenshot above.

@marla-hoggard marla-hoggard left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Well that was easy! Hopefully all of our errors are caught and re-thrown correctly so thsi doesn't bleed any unwanted error message details to the user.

@joustrich
joustrich merged commit d426029 into master Jul 20, 2026
1 check 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.

2 participants