Skip to content

Commit

Permalink
chore(docs): updating alerts & reports documentation WEBDRIVER_BASEUR…
Browse files Browse the repository at this point in the history
…L settings for docker compose (#27951)

Co-authored-by: Evan Rusackas <[email protected]>
  • Loading branch information
fisjac and rusackas authored May 15, 2024
1 parent 6232aac commit f420005
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion docker/pythonpath_dev/superset_config.py
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,7 @@ class CeleryConfig:

FEATURE_FLAGS = {"ALERT_REPORTS": True}
ALERT_REPORTS_NOTIFICATION_DRY_RUN = True
WEBDRIVER_BASEURL = "http://superset:8088/"
WEBDRIVER_BASEURL = "http://superset:8088/" # When using docker compose baseurl should be http://superset_app:8088/
# The base URL for the email report hyperlinks.
WEBDRIVER_BASEURL_USER_FRIENDLY = WEBDRIVER_BASEURL

Expand Down
2 changes: 1 addition & 1 deletion docs/docs/configuration/alerts-reports.mdx
Original file line number Diff line number Diff line change
Expand Up @@ -160,7 +160,7 @@ WEBDRIVER_OPTION_ARGS = [
]

# This is for internal use, you can keep http
WEBDRIVER_BASEURL = "http://superset:8088"
WEBDRIVER_BASEURL = "http://superset:8088" # When running using docker compose use "http://superset_app:8088'
# This is the link sent to the recipient. Change to your domain, e.g. https://superset.mydomain.com
WEBDRIVER_BASEURL_USER_FRIENDLY = "http://localhost:8088"
```
Expand Down

0 comments on commit f420005

Please sign in to comment.