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-302317: Release Java Mgmt SDK v2.0.0 #422

Merged
merged 53 commits into from
Jul 7, 2020

Conversation

arvindkrishnakumar-okta
Copy link
Contributor

@arvindkrishnakumar-okta arvindkrishnakumar-okta commented Jul 1, 2020

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

  • Bugfix
  • Enhancement
  • New Feature
  • Configuration Change
  • Versioning Change
  • Unit Test(s)
  • Documentation

arvindkrishnakumar-okta and others added 30 commits May 27, 2020 14:44
…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
@arvindkrishnakumar-okta
Copy link
Contributor Author

There is still a bunch of unused and deprecated method/classes in dev_openapi_v2.0.0_major_rel

Addressed.

Copy link
Collaborator

@laura-rodriguez laura-rodriguez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

Copy link

@justinabrokwah-okta justinabrokwah-okta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks well done

Copy link

@bryanapellanes-okta bryanapellanes-okta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Copy link
Contributor

@bdemers bdemers left a 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)

bdemers and others added 6 commits July 6, 2020 17:42
* 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>
@arvindkrishnakumar-okta
Copy link
Contributor Author

Looks good to me, but we need a few things first:

* fix the changed license headers (see suggestions)

done

* a completed migration guide (underway on another PR)

:)

* remove remaining deprecated code(I pushed a quick PR #426)

merged it, looks good

@arvindkrishnakumar-okta arvindkrishnakumar-okta merged commit 03cabad into master Jul 7, 2020
@arvindkrishnakumar-okta arvindkrishnakumar-okta deleted the dev_openapi_v2.0.0_major_rel branch December 4, 2020 06:43
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.

8 participants