Skip to content

Conversation

@TejInaco
Copy link
Contributor

@TejInaco TejInaco commented Dec 4, 2025

Currently, the error messages are insufficient for users to understand what is happening when editing JSON. Additionally, some critical validation errors are not displayed in the front end, which impacts usability and troubleshooting.

This PR does:

  1. Display all validation errors in the UI to ensure users have full visibility.
  2. Introduce an “Additional Checks” section to surface errors provided by the validation layer that were previously unused in the front end.
  3. Improve the user experience when editing JSON by providing clearer and more actionable feedback.
  4. Fix issue JSON Schema Errors are not shown #154 related to incomplete error reporting.
  5. [Updated: Depreceated] Comment out the removal of the @type key when converting a TM to TD, as this behavior caused unintended data loss.

@netlify
Copy link

netlify bot commented Dec 4, 2025

Deploy Preview for editdor ready!

Name Link
🔨 Latest commit 07fbb74
🔍 Latest deploy log https://app.netlify.com/projects/editdor/deploys/6943e8138795ba0008087116
😎 Deploy Preview https://deploy-preview-169--editdor.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Signed-off-by: Ricardo M G da Silva <[email protected]>
@TejInaco
Copy link
Contributor Author

TejInaco commented Dec 4, 2025

Example of what a user can experience:
image
image
image
Ready to re

@TejInaco TejInaco requested a review from egekorkan December 4, 2025 16:35
Copy link
Contributor

@egekorkan egekorkan left a comment

Choose a reason for hiding this comment

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

Also tried the new frontend messages. I have turned an "observable":true into observable:"true" and got JSON Schema validation failed: Unknown schema validation error.

Signed-off-by: Ricardo M G da Silva <[email protected]>
@TejInaco
Copy link
Contributor Author

Ready to be merged.

@egekorkan egekorkan merged commit d9e1c0b into eclipse:master Dec 19, 2025
7 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.

2 participants