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

Attempting to enable the production branch_type never seems to get "fixed" #208

Conversation

barrywhart
Copy link

@barrywhart barrywhart commented May 18, 2024

Fixes #207

@@ -311,6 +311,7 @@ func flattenBranchModel(rp *BranchModel, typ string) []interface{} {
"is_valid": rp.IsValid,
"use_mainbranch": rp.UseMainbranch,
"name": rp.Name,
"enabled": rp.Enabled,
Copy link
Author

@barrywhart barrywhart May 18, 2024

Choose a reason for hiding this comment

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

For production, the BitBucket API returns an enabled field. We simply need to copy it from the BranchModel to the output. In the develop case, the enabled field is not present in the response and defaults to false. NOTE: In the develop case, the enabled field is also not part of the Terraform resource schema.

In light of the above, I'm not 100% sure about this fix. It seems a little hacky. Another potential option is to handle the enabled field of production separately so it doesn't affect development. Thoughts?

@barrywhart
Copy link
Author

@DrFaust92: Do you think you will have time to review this PR soon? No particular rush, but once this is (hopefully) merged and released, all the provider bugs affecting us at Zoro will have been fixed and we'll upgrade. 😌

Thank you. 🙏🏽

@DrFaust92 DrFaust92 merged commit 8de67e9 into DrFaust92:master Jun 4, 2024
1 check passed
@barrywhart
Copy link
Author

Thank you!!

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.

Followup to #183: Attempting to enable the production branch_type never seems to get "fixed"
2 participants