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

IAdvancedContentRepositoryUserRepository: Move "Current" out to own child repository #473

Closed
HeyJoel opened this issue Jan 4, 2022 · 1 comment

Comments

@HeyJoel
Copy link
Member

HeyJoel commented Jan 4, 2022

It feels like these could be moved out to a child repository e.g. _contentRepostory.Users().Current().XYZ:

  • UpdateCurrentUserAccountAsync
  • UpdateCurrentUserPasswordAsync
  • GetCurrent
@HeyJoel HeyJoel added this to the 0.10 milestone Jan 4, 2022
HeyJoel added a commit that referenced this issue Jan 4, 2022
@HeyJoel
Copy link
Member Author

HeyJoel commented Jan 4, 2022

Fixed, will be released in v0.10. The API has a breaking change, with methods moved under the new "Current()" extension method e.g.

var user = await _contentRepository
                .Users()
                .GetCurrent()
                .AsMicroSummary()
                .ExecuteAsync();

is now:

var user = await _contentRepository
                .Users()
                .Current()
                .Get()
                .AsMicroSummary()
                .ExecuteAsync();

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant