Skip to content
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

Improve error message readability #114

Closed
bigluck opened this issue Mar 19, 2024 · 1 comment · Fixed by #117
Closed

Improve error message readability #114

bigluck opened this issue Mar 19, 2024 · 1 comment · Fixed by #117
Assignees
Labels
enhancement New feature or request

Comments

@bigluck
Copy link

bigluck commented Mar 19, 2024

The explorer does not honor the new lines on error messages.

In my case the API endpoint returned this response error:

{
    "error": "Parser exception: Invalid input <MATCH\n    (task:PhyTask)-[r1:HasPhyTaskImplementation]->(task_impl:PhyTaskCode),\n    (docker_task:PhyTask)-[r2:PhyTaskFollowBy]->(task),\n    (docker_task)-[r3:HasPhyTaskImplementation]->(docker_task_impl:PhyTaskDockerImagePull)\nWHERE\n    task_impl.pip_dependencies IS NOT NULL\nUNWIND task_impl.pip_dependencies\nRETURN>: expected rule oC_SingleQuery (line: 8, offset: 0)\n\"RETURN *\"\n ^^^^^^"
}

But the explorer makes it harder to read:

Screenshot 2024-03-19 at 09 40 41
@mewim mewim added the enhancement New feature or request label Mar 19, 2024
@mewim mewim self-assigned this Mar 19, 2024
mewim added a commit that referenced this issue Mar 20, 2024
@mewim mewim linked a pull request Mar 20, 2024 that will close this issue
@mewim
Copy link
Member

mewim commented Mar 20, 2024

I have changed the rendering of the error messages using the <pre> element to preserve the line breaks and defaults to monospace font. This should hopefully improve the readability.
Screenshot 2024-03-20 at 6 55 19 PM

Screenshot 2024-03-20 at 6 56 31 PM

mewim added a commit that referenced this issue Mar 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants