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

remove deprecated resource and datasource #1532

Merged
merged 3 commits into from
Apr 25, 2023

Conversation

duytiennguyen-okta
Copy link
Contributor

@duytiennguyen-okta duytiennguyen-okta commented Apr 18, 2023

remove deprecated resource and data source as noted in #1338
I also removed some of attribute that was removed in the resource but not the data source (data_source_okta_user.go)
These 2 test still fail TestAccOktaPolicyRuleIdpDiscovery_crud and TestAccOktaUser_customProfileAttributes

Copy link
Collaborator

@monde monde left a comment

Choose a reason for hiding this comment

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

What a liberating PR! +85 −2,382
:shipit:

@@ -7,6 +7,7 @@ import (
"github.com/hashicorp/terraform-plugin-sdk/v2/helper/resource"
)

// TODU
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is there still a TODO here?

@@ -15,6 +15,7 @@ import (
"github.com/hashicorp/terraform-plugin-sdk/v2/terraform"
)

// TODU
Copy link
Collaborator

Choose a reason for hiding this comment

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

Done?

@virgofx
Copy link
Contributor

virgofx commented Apr 19, 2023

Super excited about removing all the cruft!

@duytiennguyen-okta duytiennguyen-okta merged commit b3942aa into v4.0.0 Apr 25, 2023
@duytiennguyen-okta duytiennguyen-okta deleted the OKTA-592057-remove-deprecated-resource branch April 25, 2023 15:13
@virgofx
Copy link
Contributor

virgofx commented Apr 25, 2023

Hey @duytiennguyen-okta Any info on when we can expect a 4.x release? Would be happy to test.

@duytiennguyen-okta
Copy link
Contributor Author

@virgofx It should be release by the end of the week

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.

3 participants