eschutho commented on code in PR #36126:
URL: https://github.com/apache/superset/pull/36126#discussion_r2548563405
##########
superset/utils/screenshots.py:
##########
@@ -16,23 +16,48 @@
# under the License.
from __future__ import annotations
+import base64
import logging
+from datetime import datetime
+from enum import Enum
from io import BytesIO
-from typing import TYPE_CHECKING
+from typing import cast, TYPE_CHECKING, TypedDict
-from flask import current_app
+from flask import current_app as app
+from superset import feature_flag_manager, thumbnail_cache
+from superset.exceptions import ScreenshotImageNotAvailableException
+from superset.extensions import event_logger
from superset.utils.hashing import md5_sha_from_dict
from superset.utils.urls import modify_url_query
from superset.utils.webdriver import (
ChartStandaloneMode,
DashboardStandaloneMode,
- WebDriverProxy,
+ WebDriver,
+ WebDriverPlaywright,
+ WebDriverSelenium,
WindowSize,
)
logger = logging.getLogger(__name__)
+# Import Playwright availability and install message
+try:
+ from superset.utils.webdriver import (
+ PLAYWRIGHT_AVAILABLE,
+ PLAYWRIGHT_INSTALL_MESSAGE,
+ )
+except ImportError:
+ PLAYWRIGHT_AVAILABLE = False
+ PLAYWRIGHT_INSTALL_MESSAGE = "Playwright module not found"
Review Comment:
This suggestion is not related to this PR.
--
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]
