-
Notifications
You must be signed in to change notification settings - Fork 136
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-302317: Release Java Mgmt SDK v2.0.0 #422
Conversation
…serWithApplicationTest OKTA-279039: Fix flaky ApplicationsIT associateUserWithApplication test
…onsTest OKTA-279039: Fix flaky GroupsIT groupUserOperationsTest test
* openapi csr rename update
* revert csr model rename
* open api spec update
Addressed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks well done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
integration-tests/src/test/groovy/com/okta/sdk/tests/it/CrudTestSupport.groovy
Outdated
Show resolved
Hide resolved
integration-tests/src/test/groovy/com/okta/sdk/tests/it/FactorsIT.groovy
Outdated
Show resolved
Hide resolved
integration-tests/src/test/groovy/com/okta/sdk/tests/it/GroupsIT.groovy
Outdated
Show resolved
Hide resolved
integration-tests/src/test/groovy/com/okta/sdk/tests/it/UsersIT.groovy
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, but we need a few things first:
- fix the changed license headers (see suggestions)
- a completed migration guide (underway on another PR)
- remove remaining deprecated code(I pushed a quick PR Remove deprecated code #426)
* Remove deprecated code * fix typo
…stSupport.groovy Co-authored-by: Brian Demers <bdemers@apache.org>
…sIT.groovy Co-authored-by: Brian Demers <bdemers@apache.org>
…T.groovy Co-authored-by: Brian Demers <bdemers@apache.org>
…IT.groovy Co-authored-by: Brian Demers <bdemers@apache.org>
done
:)
merged it, looks good |
Issue(s)
OKTA-302317
Description
Release Java Mgmt SDK
v2.0.0
with Open API v2.0.0 upgrades.ℹ️ Commits beginning 07/01 are the ones that need to be reviewed as part of this PR. Changes shown in this PR prior to 07/01 commit were already reviewed by individual PRs that were merged to this development branch.
Category