Skip to content

Deprecate old list methods that do not support pagination#136

Merged
lbalmaceda merged 1 commit intomasterfrom
deprecate-old-list
Jul 12, 2018
Merged

Deprecate old list methods that do not support pagination#136
lbalmaceda merged 1 commit intomasterfrom
deprecate-old-list

Conversation

@lbalmaceda
Copy link
Copy Markdown
Contributor

@lbalmaceda lbalmaceda commented Jul 4, 2018

Based on #135 (merge that one first and rebase).

Closes #134

The following methods are deprecated in favor of alternatives that support Pagination

  • ClientGrantsEntity#list()
  • ConnectionsEntity#list(ConnectionFilter filter)
  • GrantsEntity#list(String userId)
  • ResourceServerEntity#list()

@lbalmaceda lbalmaceda added this to the v1-Next milestone Jul 4, 2018
@cocojoe
Copy link
Copy Markdown
Member

cocojoe commented Jul 5, 2018

@lbalmaceda can you show clearly in description which methods are being deprecated and their replacement. We should always show Public API changes in description. Thx

@lbalmaceda lbalmaceda force-pushed the deprecate-old-list branch from a73eed3 to dd1b032 Compare July 11, 2018 23:05
@cocojoe cocojoe changed the base branch from master to fix-get-connections July 12, 2018 08:22
@cocojoe cocojoe changed the base branch from fix-get-connections to master July 12, 2018 08:22
Copy link
Copy Markdown
Member

@cocojoe cocojoe left a comment

Choose a reason for hiding this comment

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

I changed base to the other PR to review, then changed back, looks good.

@lbalmaceda lbalmaceda force-pushed the deprecate-old-list branch from dd1b032 to 336aec0 Compare July 12, 2018 13:22
@lbalmaceda lbalmaceda merged commit e38684d into master Jul 12, 2018
@lbalmaceda lbalmaceda deleted the deprecate-old-list branch July 12, 2018 19:25
@lbalmaceda lbalmaceda modified the milestones: v1-Next, 1.8.0 Jul 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants