-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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.
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
doc(ingestion/tableau): Fix rendering ingestion quickstart guide #7808
doc(ingestion/tableau): Fix rendering ingestion quickstart guide #7808
Conversation
Please fix the bad image urls. |
<img width="75%" alt="tableau site role" src="https://raw.githubusercontent.com/datahub-project/static-assets-fork/main/imgs/guides/tableau/tableau-site-role.png"/> | ||
</p> | ||
<p align="center"> | ||
<img width="75%" alt="tableau site role" src="https://raw.githubusercontent.com/datahub-project/static-assets/main/imgs/guides/tableau/tableau-site-role.png"/> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks like this image was not redacted
@@ -60,7 +60,7 @@ If you do not see the Ingestion tab, please contact your DataHub admin to grant | |||
6. Select Tableau | |||
|
|||
<p align="center"> | |||
<img width="70%" alt="Select Tableau from the options" src="https://raw.githubusercontent.com/datahub-project/static-assets-fork/main/imgs/guides/tableau/tableau-new-ingestion-source.png"/> | |||
<img width="70%" alt="Select Tableau from the options" src="https://raw.githubusercontent.com/datahub-project/static-assets/main/imgs/guides/tableau/tableau-new-ingestion-source.png"/> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks like most of these image links are broken https://docs-website-1ua8gifbf-acryldata.vercel.app/docs/quick-ingestion-guides/powerbi/setup
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @hsheth2
Please verify again, I have corrected the links, however whenever from git I am trying to download or open the image, I am getting below error
The endpoint raw.githubusercontent.com
is not rechable.
@@ -20,15 +20,15 @@ In order to configure ingestion from PowerBI, you'll first have to ensure you ha | |||
e. On `Register an application` window fill the `Name` of application says `powerbi-app-connector` and keep other default as is | |||
|
|||
<p align="center"> | |||
<img width="75%" alt="app_registration" src="https://raw.githubusercontent.com/main/static-assets-fork/main/imgs/guides/powerbi/app-registration.png"/> | |||
<img width="75%" alt="app_registration" src="https://raw.githubusercontent.com/main/static-assets/main/imgs/guides/powerbi/app-registration.png"/> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
these links still don't work - please fix this one and the others
<img width="75%" alt="app_registration" src="https://raw.githubusercontent.com/main/static-assets/main/imgs/guides/powerbi/app-registration.png"/> | |
<img width="75%" alt="app_registration" src="https://raw.githubusercontent.com/datahub-project/static-assets/main/imgs/guides/powerbi/app-registration.png"/> |
No description provided.