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

Jump to bottom

Tableau descriptions and URLs #9116

Closed
pmbrull opened this issue Dec 2, 2022 · 0 comments · Fixed by #9241
Closed

Tableau descriptions and URLs #9116

pmbrull opened this issue Dec 2, 2022 · 0 comments · Fixed by #9241
Assignees
@pmbrull
Labels
bug Something isn't working Ingestion

Comments

@pmbrull
Copy link
Collaborator

pmbrull commented Dec 2, 2022

  • In the teableau connector we are not picking the descriptions. We should be able to gather them from Workbook Details > About.
  • Double-check that the URL is being built properly during the ingestion. For example, Workbooks with spaces get quoted as %20 while tableau just removes the spaces.
@pmbrull pmbrull self-assigned this Dec 2, 2022
@pmbrull pmbrull added bug Something isn't working Ingestion labels Dec 2, 2022
@pmbrull pmbrull moved this to 🏗 In progress in Release 0.13.1 Dec 2, 2022
@pmbrull pmbrull moved this from 🏗 In progress to 🔖 Ready in Release 0.13.1 Dec 2, 2022
@pmbrull pmbrull removed this from Release 1.0 Dec 11, 2022
@pmbrull pmbrull moved this to 🏗 In progress in Release 0.13.1 Dec 11, 2022
@pmbrull pmbrull mentioned this issue Dec 11, 2022
Merged
9 tasks
@pmbrull pmbrull moved this from 🏗 In progress to 👀 In review in Release 0.13.1 Dec 11, 2022
pmbrull added a commit that referenced this issue Dec 13, 2022
@pmbrull
Fix #9116 & #8284 - Clean tableau source, fix ownership, add descript…
c75ba75 
…ion and SSL verification (#9241)

Fix #9116 & #8284 - Clean tableau source, fix ownership, add description and SSL verification (#9241)
Repository owner moved this from 👀 In review to 🔖 Ready in Release 0.13.1 Dec 13, 2022
OnkarVO7 pushed a commit to OnkarVO7/OpenMetadata that referenced this issue Dec 15, 2022
@pmbrull @OnkarVO7
Fix open-metadata#9116 & open-metadata#8284 - Clean tableau source, f…
c9df522 
…ix ownership, add description and SSL verification (open-metadata#9241)

Fix open-metadata#9116 & open-metadata#8284 - Clean tableau source, fix ownership, add description and SSL verification (open-metadata#9241)
@harshach harshach moved this from 🔖 Ready to ✅ Done in Release 0.13.1 Jan 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pmbrull pmbrull

Labels
bug Something isn't working Ingestion
Projects
No open projects
Release 0.13.1
Status: ✅ Done
Milestone
No milestone
1 participant
@pmbrull