etr2460 commented on issue #8747: feat: emphasis on the database engine error 
message
URL: 
https://github.com/apache/incubator-superset/pull/8747#issuecomment-561777971
 
 
   I've considered similar fixes before, but have punted because instead of 
overloading the single string error message with metadata/styling, I think it'd 
be a better path forward to make the error message a structured payload, 
something like:
   ```
   {
     title: "error title",
     message: "message",
     link: "optional link",
     stacktrace: "optional stacktrace", 
   }
   ```
   
   Then we could always render it appropriately on the front end.
   
   Note that we could still store the error in the db as a string, but just 
pass a structured error back to the frontend

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to