eschutho commented on code in PR #30274:
URL: https://github.com/apache/superset/pull/30274#discussion_r1761600651


##########
superset/config.py:
##########
@@ -1427,11 +1431,6 @@ def EMAIL_HEADER_MUTATOR(  # pylint: 
disable=invalid-name,unused-argument
 SLACK_API_TOKEN: Callable[[], str] | str | None = None
 SLACK_PROXY = None
 
-# Whether Superset should use Slack avatars for users.
-# If on, you'll want to add "https://avatars.slack-edge.com"; to the list of 
allowed
-# domains in your TALISMAN_CONFIG
-SLACK_ENABLE_AVATARS = False
-

Review Comment:
   Since this hasn't been released in an official version, I think this 
breaking change is ok.



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to