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.

Sign up for GitHub

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

Jump to bottom

Tgoodsell tempus 1734 #1756

Merged
merged 19 commits into from
Oct 10, 2023
Merged

Tgoodsell tempus 1734 #1756

merged 19 commits into from
Oct 10, 2023

Conversation

duytiennguyen-okta
Copy link
Contributor

@duytiennguyen-okta duytiennguyen-okta commented Oct 10, 2023 •
edited by monde
Loading

This brings in @tgoodsell-tempus 's #1734

This is a new resource to allow the assignment of admin roles on OAuth Clients in Okta. Note this feature may be behind a feature flag in your org depending on it's version and other settings.

API reference: https://developer.okta.com/docs/api/openapi/okta-oauth/oauth/tag/RoleAssignment/

Adding to the manual SDK for now since the Okta Golang SDK does not yet have these API calls supported.

Ref: #1662

tgoodsell-tempus and others added 19 commits September 21, 2023 01:39
@tgoodsell-tempus
create sdk custom API functions for client role assignment, including…
d29b614 
… custom structs for operations
@tgoodsell-tempus
use cloned request executor
750bac6
@tgoodsell-tempus
add code for AppOAuthRoleAssignment resource
17613d9
@tgoodsell-tempus
add resource to provider and sort list
225cb32
@tgoodsell-tempus
start adding test code
cbb7acc
@tgoodsell-tempus
add basic fixtures
2fa4484
@tgoodsell-tempus
add test checks
4e3d801
@tgoodsell-tempus
add back Configure function, switch optional fields in the JSON to po…
9589256 
…inters to ensure proper null handling
@tgoodsell-tempus
add test fixtures for CUSTOM
b8bdea3
@tgoodsell-tempus
fix the validation handler check
e9ca923
@tgoodsell-tempus
add website docs and fix test
82cf0ec
@tgoodsell-tempus
fix test
e1b9556
@tgoodsell-tempus
add import test
057985d
@tgoodsell-tempus
consolidate test
17893bf
@tgoodsell-tempus
implement ImportState
1346623
@tgoodsell-tempus
fix ID setup for import test
c8ebc6b
@tgoodsell-tempus
add docs for import
4af3633
@duytiennguyen-okta
Merge remote-tracking branch 'tgoodsell-tempus/okta-app-role-assignme…
ba95fa4 
…nt' into tgoodsell-tempus_1734
@duytiennguyen-okta
update example
ad2d6bb
@duytiennguyen-okta duytiennguyen-okta merged commit 769bc41 into master Oct 10, 2023
2 checks passed
@duytiennguyen-okta duytiennguyen-okta deleted the tgoodsell-tempus_1734 branch October 10, 2023 17:49
@monde monde mentioned this pull request Oct 17, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@monde monde monde approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@duytiennguyen-okta @monde @tgoodsell-tempus