-
Notifications
You must be signed in to change notification settings - Fork 207
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
Support Custom Group Profile Schema #851
Conversation
@bogdanprodan-okta @monde QQ for you regarding this PR. I see the version of the SDK you're currently using is some custom version of the Would y'all mind weighing in on how we should proceed here? I noticed that if I manually swapped to the normal release, there were type issues and other things raised, so it's deviated further than I expected. |
@ymylei currently, a generator that is used to generate SDK's codebase is not very flexible, so I keep some manual changes in a separate branch. This is necessary for the correct behavior of the provider. I've created PR #852 that contains all the necessary fields. |
Cheers! Much appreciated! |
…e read back into state
@bogdanprodan-okta @monde Pretty sure this is ready to go, notifying you since my tests were producing some errors but passing. Based on the messages I suspect it's just weird behavior from the |
Updated test results:
|
@ymylei I would suggest adding
|
@bogdanprodan-okta Good shout! I used the index values and was able to streamline the test. Updated results here:
|
Results of
|
PR is designed to:
custom_profile_attributes
fromokta_user
and implement inokta_group