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

okta_app_auto_login enhancements #164

Merged
merged 4 commits into from
Nov 10, 2020
Merged

okta_app_auto_login enhancements #164

merged 4 commits into from
Nov 10, 2020

Conversation

isometry
Copy link
Contributor

@isometry isometry commented Aug 5, 2020

  • Enhance okta_app_auto_login resource by adding support for user and group assignment, and allowing the user_name_template* attributes to be overridden.
  • Add missing arguments to okta_app_auto_login documentation.
  • Refactor buildSchema() as variadic to simplify schema logic.

@hrzbrg
Copy link

hrzbrg commented Aug 12, 2020

Hey, thanks for adding all these to the okta_app_auto_login, we would really love to see it in the provider as soon as possible, as we have teams requesting addition of such apps and currently it is still a manual task. In the future they can just hand in PRs for this :)

@noinarisak noinarisak added the needs-investigation Needs further investigation label Aug 31, 2020
* Enhance `okta_app_auto_login` resource by adding support for user
  and group assignment, and allowing the `user_name_template*`
  attributes to be overridden.
* Add missing arguments to `okta_app_auto_login` documentation.
* Refactor `buildSchema()` as variadic to simplifying schema logic.
@bryantbiggs
Copy link
Contributor

any thoughts @bogdanprodan-okta / @noinarisak ?

@isometry
Copy link
Contributor Author

isometry commented Nov 3, 2020

We've been using the changes in this PR as a critical part of our Okta configuration since we migrated in August… I'll feel far more comfortable once this is merged and we no longer have to maintain a custom provider mirror incorporating these changes.

@bogdanprodan-okta
Copy link
Contributor

bogdanprodan-okta commented Nov 5, 2020

Hi @isometry! Thanks for the PR! Could you please add or edit a test case for this functionality?

PS I've already tested it on a test company with updated test cases and it's working as expected

@bogdanprodan-okta
Copy link
Contributor

@noinarisak I think this PR is ready to be merged.

@noinarisak noinarisak added enhancement Asking for new behavior or feature and removed needs-investigation Needs further investigation labels Nov 5, 2020
@noinarisak noinarisak self-assigned this Nov 5, 2020
@bogdanprodan-okta bogdanprodan-okta merged commit 2f184c4 into okta:master Nov 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Asking for new behavior or feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants