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

Updating the ListUsers() Implementation #77

Merged
merged 9 commits into from Jun 19, 2019

Conversation

@hiranya911
Copy link
Member

commented Jun 17, 2019

  • Cleaned up the public API according to go/dotnet-user-mgt
  • Added tests to cover different ways of iterating through user accounts

Follow up to #58

hiranya911 added some commits Jun 15, 2019

@hiranya911 hiranya911 requested a review from bklimt Jun 17, 2019

@bklimt

bklimt approved these changes Jun 19, 2019

Copy link

left a comment

LGTM

@hiranya911 hiranya911 assigned hiranya911 and unassigned bklimt Jun 19, 2019

@hiranya911 hiranya911 merged commit 6061bec into master Jun 19, 2019

3 checks passed

cla/google All necessary CLAs are signed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@hiranya911 hiranya911 deleted the hkj-list-cleanup branch Jun 19, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.