You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@superset.apache.org by GitBox <gi...@apache.org> on 2019/11/27 18:56:10 UTC

[GitHub] [incubator-superset] rusackas commented on issue #8681: Only list tables in DB test if API returns them (which it doesn't)

rusackas commented on issue #8681: Only list tables in DB test if API returns them (which it doesn't)
URL: https://github.com/apache/incubator-superset/pull/8681#issuecomment-559211901
 
 
   > That original feature that shows the table name is a weird artifact from super early (like during the original hackathon) debugging. We could deprecate that.
   
   Ok... given that, I just took that code out instead of wrapping it in a conditional. If we need to display tables, I'm guessing we'll go a different (e.g. React) direction with it anyway, rather than building on the jQuery stuff :)

----------------------------------------------------------------
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