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.

Sign up for GitHub

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

Jump to bottom

IDomainRepository.WithExecutionContext: Support passing in IUserContext #483

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

IDomainRepository.WithExecutionContext: Support passing in IUserContext #483

HeyJoel opened this issue Jan 27, 2022 · 1 comment
Labels
breaking enhancement
Milestone
0.10

Comments

@HeyJoel
Copy link
Member

HeyJoel commented Jan 27, 2022

We can currently elevate permissions by passing in a new execution context via IDomainRepository.WithExecutionContext, but if you're elevating permissions you probably just want to pass the IUserContext and not have to deal with the other execution context properties.

In doing this we'd probably want to rename the existing feature to "WithContext" and support both contexts with overloads. This would be a breaking change, but a simple rename.
@HeyJoel HeyJoel added this to the 0.10 milestone Jan 27, 2022
HeyJoel added a commit that referenced this issue Jan 28, 2022
@HeyJoel
Fixes #483 - IDomainRepository.WithExecutionContext: Support passing …
9e5e392 
…in IUserContext
@HeyJoel
Copy link
Member Author

HeyJoel commented Jan 28, 2022

Fixed, will be release in v0.10

@HeyJoel HeyJoel closed this as completed Jan 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
breaking enhancement
Projects
None yet
Milestone
0.10
Development

No branches or pull requests
1 participant
@HeyJoel