[GitHub] [incubator-superset] etr2460 commented on issue #8747: feat: emphasis on the database engine error message

[GitBox <gi...@apache.org>]

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:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org