hughhhh commented on code in PR #29687: URL: https://github.com/apache/superset/pull/29687#discussion_r1707420370
########## superset/config.py: ########## @@ -320,6 +320,8 @@ def _try_json_readsha(filepath: str, length: int) -> str | None: # ex: http://localhost:8080/swagger/v1 FAB_API_SWAGGER_UI = True +FAB_ADD_SECURITY_API = True Review Comment: @dpgaspar did we make and update in flask app builder to make this default True? When I recently pulled these were getting `KeyError: FAB_ADD_SECURITY_API not found` Also I saw that flask app builder has this default True so to allow these test to run I just made this value consistent with what I saw [upstream](https://github.com/dpgaspar/Flask-AppBuilder/blob/2d527aae09eed50fd52c6bf886a2970adb225428/examples/crud_rest_api/config.py#L5) Can you verify that this is what's happening? -- 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. To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org For additional commands, e-mail: notifications-h...@superset.apache.org
