Skip to content

Conversation

@yoodan93
Copy link
Contributor

@yoodan93 yoodan93 commented Sep 12, 2025

Which problem is this PR solving?

No issue but something i noticed while migrating boards to Terraform.

We were missing query_annotation import.

Short description of the changes

Adding ImportState for query_annotation resource type

How to verify that this has the expected result

Adding unit test.

The import state follows similar pattern to the query resource.

@yoodan93 yoodan93 added the feature This issue wants to add new functionality. label Sep 15, 2025
@yoodan93 yoodan93 changed the title feat(query-annotations): Add ImportState to QuerAnnotation resource feat(query-annotations): Add ImportState to QueryAnnotation resource Sep 15, 2025
@yoodan93 yoodan93 marked this pull request as ready for review September 15, 2025 16:28
@yoodan93 yoodan93 requested a review from a team as a code owner September 15, 2025 16:28
Copy link
Contributor

@jharley jharley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this! Nice work

A few things ahead of a stamp:

  1. the documentation for the query_annotation resource needs to be updated to show that it does now support imports
  2. the title of the PR isn't quite right: the prefix should be feat(r/query_annotation): (as it's affecting the query_annotation resource. The description itself should likely be a bit more consumer focused (i.e. "add import support")
  3. you should add the 'feature' label to this PR as it is used to generate the changelog for releases

@yoodan93 yoodan93 changed the title feat(query-annotations): Add ImportState to QueryAnnotation resource feat(r/query_annotation): Add ImportState to QueryAnnotation resource Sep 15, 2025
@yoodan93
Copy link
Contributor Author

Thank you @jharley , let me get those changes for you!

@yoodan93 yoodan93 changed the title feat(r/query_annotation): Add ImportState to QueryAnnotation resource feat(r/query_annotation): Add import support for Query Annotation resource. Sep 15, 2025
@jharley jharley changed the title feat(r/query_annotation): Add import support for Query Annotation resource. feat(r/query_annotation): add import support Sep 15, 2025
@yoodan93 yoodan93 merged commit 4bbba01 into main Sep 15, 2025
7 checks passed
@yoodan93 yoodan93 deleted the dyoo/query-annotation-import branch September 15, 2025 17:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature This issue wants to add new functionality.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants