Use source_info.main_url instead of crafting workbook url in tableau …

…extractor (#941) * Use source_info.main_url instead of crafting workbook url in tableau extractor * bump version --------- Co-authored-by: Scott Ssuyi Huang <[email protected]>
MetaphorData · Aug 7, 2024 · 62ce241 · 62ce241
1 parent fa40d88
commit 62ce241
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/metaphor/tableau/extractor.py b/metaphor/tableau/extractor.py
@@ -525,7 +525,11 @@ def _parse_workbook_query_response(
                         custom_sql_source.account if custom_sql_source else None
                     ),
                     source_datasets=source_datasets or None,
-                    url=f"{self._base_url}/workbooks/{workbook.vizportalUrlId}",
+                    url=(
+                        dashboard.source_info.main_url
+                        if dashboard.source_info
+                        else None
+                    ),
                 ),
                 entity_upstream=(
                     EntityUpstream(source_entities=source_datasets)

diff --git a/pyproject.toml b/pyproject.toml
@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "metaphor-connectors"
-version = "0.14.70"
+version = "0.14.71"
 license = "Apache-2.0"
 description = "A collection of Python-based 'connectors' that extract metadata from various sources to ingest into the Metaphor app."
 authors = ["Metaphor <[email protected]>"]