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

Deprecate users and groups field in the okta_app_* data sources #497

Merged
merged 3 commits into from
Jul 2, 2021

Conversation

ymylei
Copy link
Contributor

@ymylei ymylei commented May 31, 2021

Adds deprecations to these fields ahead of their removal. Requires the replacement data sources below (if deprecation messages are not going to be altered or functionality not replaced).

Replacement data sources:

@ymylei ymylei changed the title [WIP] Deprecate users and groups field in the okta_app data source Deprecate users and groups field in the okta_app data source May 31, 2021
@ymylei ymylei marked this pull request as ready for review May 31, 2021 15:59
@ymylei ymylei changed the title Deprecate users and groups field in the okta_app data source Deprecate users and groups field in the okta_app_* data sources May 31, 2021
@bogdanprodan-okta
Copy link
Contributor

Hi @ymylei! Because we are planning to release 4.x.x version, we are going to remove all the user and groups fields from all the app related resources, this PR is redundant.

@ymylei
Copy link
Contributor Author

ymylei commented Jun 28, 2021

@bogdanprodan-okta I believe that is a mistake on your part. According to the deprecation best practices from Hashi (https://www.terraform.io/docs/extend/best-practices/deprecations.html) we should be releasing a "deprecated" release before you remove anything in a major cut.

@bogdanprodan-okta bogdanprodan-okta linked an issue Jun 30, 2021 that may be closed by this pull request
@bogdanprodan-okta bogdanprodan-okta merged commit 2487079 into okta:master Jul 2, 2021
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.

okta_app_*: users and groups field deprecation/removal
2 participants