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

[fix] doc for importing table #525

Merged
merged 2 commits into from
Apr 28, 2021
Merged

Conversation

yohei1126
Copy link
Contributor

@yohei1126 yohei1126 commented Apr 27, 2021

The target table id is missing on the current doc. This will fix the issue.

Test Plan

  • acceptance tests

References

@yohei1126 yohei1126 requested a review from a team as a code owner April 27, 2021 02:24
@yohei1126 yohei1126 requested a review from alldoami April 27, 2021 02:24
@yohei1126
Copy link
Contributor Author

@ryanking @alldoami Would you check this?

@alldoami
Copy link
Contributor

/ok-to-test sha=0668ba6

@alldoami
Copy link
Contributor

Also, fyi @yohei1126, rk doesn't work at CZI anymore, so it'll be me mainly reviewing PRs until Snowflake starts to take more ownership of it.

@yohei1126
Copy link
Contributor Author

OK thanks. updated the branch.

@alldoami
Copy link
Contributor

/ok-to-test sha=ce79745

@github-actions
Copy link

Integration tests success for 0668ba6

@alldoami alldoami merged commit bbe0577 into Snowflake-Labs:main Apr 28, 2021
@github-actions
Copy link

Integration tests success for ce79745

@yohei1126 yohei1126 deleted the fix-doc-table branch April 28, 2021 00:34
anton-chekanov pushed a commit to anton-chekanov/terraform-provider-snowflake that referenced this pull request Jan 25, 2022
daniepett pushed a commit to daniepett/terraform-provider-snowflake that referenced this pull request Feb 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants