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.

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

Get all tables in the BigQuery #2458

Merged
merged 3 commits into from
Jul 29, 2018

Conversation

kyoshidajp
Copy link
Member

Summary

When using BigQuery data source, all tables are not listed in the query edit page.

Thre result will be cached, so there will no be a problem.

Changes

Add pageToken parameter into the getting tables request if result data has nextPageToken field.

@kyoshidajp
Copy link
Member Author

CircleCI failed tests will succeed if #2457 is merged.

@kyoshidajp
Copy link
Member Author

I believe this fix the issue. Could anyone review this PR?

@RichardLitt
Copy link

Hey @kyoshidajp! Sorry for the delay. We're a bit behind on issues, especially because of the v4 release. We'll get to this when we have some free time.

@kyoshidajp
Copy link
Member Author

Hi @RichardLitt. Thank you for your reply. I got it.

@arikfr arikfr merged commit dd2f24f into getredash:master Jul 29, 2018
@arikfr
Copy link
Member

arikfr commented Jul 29, 2018

Thank you, @kyoshidajp!

Apologies for the long wait.

@kyoshidajp kyoshidajp deleted the get_all_tables_in_bq branch July 29, 2018 22:30
@arikfr arikfr mentioned this pull request Aug 6, 2018
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants