Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(tableau): fixes wrong argument when reauthenticating #12216

Merged
merged 10 commits into from
Dec 24, 2024

Merge branch 'master' into feature/cus-3546-fix-tableau-authentication

7a03ddf
Select commit
Loading
Failed to load commit list.
Merged

fix(tableau): fixes wrong argument when reauthenticating #12216

Merge branch 'master' into feature/cus-3546-fix-tableau-authentication
7a03ddf
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Dec 24, 2024 in 1s

23.80% of diff hit (target 74.94%)

View this Pull Request on Codecov

23.80% of diff hit (target 74.94%)

Annotations

Check warning on line 854 in metadata-ingestion/src/datahub/ingestion/source/tableau/tableau.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/tableau/tableau.py#L845-L854 

Added lines #L845 - L854 were not covered by tests

Check warning on line 856 in metadata-ingestion/src/datahub/ingestion/source/tableau/tableau.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/tableau/tableau.py#L856 

Added line #L856 was not covered by tests

Check warning on line 913 in metadata-ingestion/src/datahub/ingestion/source/tableau/tableau.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/tableau/tableau.py#L911-L913 

Added lines #L911 - L913 were not covered by tests

Check warning on line 916 in metadata-ingestion/src/datahub/ingestion/source/tableau/tableau.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/tableau/tableau.py#L916 

Added line #L916 was not covered by tests

Check warning on line 918 in metadata-ingestion/src/datahub/ingestion/source/tableau/tableau.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/tableau/tableau.py#L918 

Added line #L918 was not covered by tests
View more details on Codecov