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 2020/07/21 00:33:03 UTC

[GitHub] [incubator-superset] cyw233 commented on pull request #9825: feat: Add Excel sheet upload

cyw233 commented on pull request #9825:
URL: https://github.com/apache/incubator-superset/pull/9825#issuecomment-661490394


   Hi @blcksrx and @villebro thanks for bringing this feature to Superset! Today when I first tried this feature it gave me this error: "AttributeError: 'SupersetSecurityManager' object has no attribute 'database_access'",  but when I changed ` security_manager.database_access(database)` to `security_manager.can_access_database(database)` it worked. I'm not quite understand why the change works, so could you guys help me with this? Cheers!
   ![Screen Shot 2020-07-21 at 10 25 52 am](https://user-images.githubusercontent.com/49756587/87999093-315f6380-cb3d-11ea-95c8-3be5eacb00f1.png)
   <img width="656" alt="Screen Shot 2020-07-21 at 10 25 09 am" src="https://user-images.githubusercontent.com/49756587/87999099-358b8100-cb3d-11ea-907a-9367a2753d18.png">
   
   


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



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