Skip to content

Conversation

@pippitt
Copy link
Contributor

@pippitt pippitt commented Oct 28, 2025

Implemented import functionality on resourcegithub_organization_custom_properties
renamed file to match others (s to z)
add some testing

Resolves #2828


Before the change?

  • Could not import the organization_custom_properties resource
    local example
$ terraform import github_organization_custom_properties.point_of_contact Point-of-Contact
Acquiring state lock. This may take a few moments...
github_organization_custom_properties.point_of_contact: Importing from ID "Point-of-Contact"...
github_organization_custom_properties.point_of_contact: Import prepared!
  Prepared github_organization_custom_properties for import
github_organization_custom_properties.point_of_contact: Refreshing state... [id=Point-of-Contact]
╷
│ Error: GET https://api.github.com/orgs/missionlane/properties/schema/: 404 Not Found []
│
│
╵

Releasing state lock. This may take a few moments...

After the change?

  • Can import organization_custom_properties resource
    local example from updated
$ terraform import github_organization_custom_properties.point_of_contact Point-of-Contact
Acquiring state lock. This may take a few moments...
github_organization_custom_properties.point_of_contact: Importing from ID "Point-of-Contact"...
github_organization_custom_properties.point_of_contact: Import prepared!
  Prepared github_organization_custom_properties for import
github_organization_custom_properties.point_of_contact: Refreshing state... [id=Point-of-Contact]

Import successful!

The resources that were imported are shown above. These resources are now in
your Terraform state and will henceforth be managed by Terraform.

Pull request checklist

  • Schema migrations have been created if needed (example) - not needed
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been reviewed and added / updated if needed (for bug fixes / features) -current docs have correct import

Does this introduce a breaking change?

Please see our docs on breaking changes to help!

  • Yes
  • No

@pippitt
Copy link
Contributor Author

pippitt commented Oct 28, 2025

borrowed from existing tests, working on the linting error

@pippitt pippitt force-pushed the fix-organization_custom_properties-import branch from 87c115c to 86398a7 Compare October 28, 2025 19:29
renamed file to match others
add some testing
@pippitt pippitt force-pushed the fix-organization_custom_properties-import branch from 86398a7 to b07b3c3 Compare October 29, 2025 18:01
@nickfloyd nickfloyd added the Type: Bug Something isn't working as documented label Oct 29, 2025
@nickfloyd nickfloyd moved this from Backlog to In Review in Terraform Provider Oct 29, 2025
Copy link
Contributor

@nickfloyd nickfloyd left a comment

Choose a reason for hiding this comment

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

❤️

@nickfloyd nickfloyd merged commit 989ca6b into integrations:main Oct 29, 2025
3 checks passed
@github-project-automation github-project-automation bot moved this from In Review to Done in Terraform Provider Oct 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: Bug Something isn't working as documented

Projects

Development

Successfully merging this pull request may close these issues.

[BUG]: organization_custom_properties import

2 participants