common-fate / terraform-provider-commonfate

The Common Fate Terraform Provider
MIT License
2 stars 2 forks source link

Deprecated field tryextendafter should not be a required field in access workflow #100

Closed ckluy31 closed 2 months ago

ckluy31 commented 2 months ago

UAT testing was leading to error Error: Provider produced inconsistent result after apply

This PR allows tryExtendAfter to default to 0 if not set, since it is being deprecated in favour of ExtensionConditions. tryExtendAfter does not do anything when set in the teams code, but needs to exist in terraform to not cause breaking changes for customers.

changeset-bot[bot] commented 2 months ago

⚠️ No Changeset found

Latest commit: e08c1b102572a46fc923a7a1664a31a3602d5b8a

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR