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

fix: display only the error message for sql-editor #3174

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

alecsci
Copy link
Member

@alecsci alecsci commented Nov 20, 2024

fix #3132

Copy link
Member Author

@alecsci alecsci left a comment

Choose a reason for hiding this comment

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

For the users this is better, for the devs this is worse because we lose the stack-trace, so maybe we should have at the end of the alert message a field for the stack-trace ?

@alecsci alecsci changed the title fix: display only the error message for issue #3132 fix: display only the error message for sql-editor Nov 20, 2024
@TheSCREWEDSoftware
Copy link
Contributor

I would say the "stack" error makes more sense if you're in developer/debug mode

@alecsci
Copy link
Member Author

alecsci commented Nov 21, 2024

Not sure if it's worth doing that much implementation to hide some technical info.
I would like to keep the error component dumb as possible.

We could show both information, but put first the error message, and then the last field can be the stack trace.
@FrancescoBorzi wdyt?

@Helias
Copy link
Member

Helias commented Nov 21, 2024

We could show both but in a separate way, so the user can focus on the error message first and then if it's useful can also read the stack.

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.

[Enhancement] - Simplification of SQL Errors
3 participants