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/10/03 05:28:12 UTC

[GitHub] [incubator-superset] tranv94 commented on pull request #11146: fix: Adding authlib as a requirement package

tranv94 commented on pull request #11146:
URL: https://github.com/apache/incubator-superset/pull/11146#issuecomment-703049850


   > Many people do not need authlib. This is clearly an optional dependency.
   
   Hmm, do you propose we should append to the (under `Custom OAuth2 Configuration`) [documentation](https://superset.apache.org/docs/installation/configuring-superset)? Feels weird to show example code that includes an optional package with no hint that it is external.


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