Skip to content
This repository has been archived by the owner on Jul 31, 2024. It is now read-only.

new resource based configuration changes #509

Merged
merged 13 commits into from
Nov 29, 2016
Merged

new resource based configuration changes #509

merged 13 commits into from
Nov 29, 2016

Conversation

brockallen
Copy link
Member

@brockallen brockallen commented Nov 29, 2016

@dnfclas
Copy link

dnfclas commented Nov 29, 2016

Hi @brockallen, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!

The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.

TTYL, DNFBOT;

@brockallen brockallen added this to the RC4 milestone Nov 29, 2016
@brockallen brockallen changed the title Config changes new resource based configuration changes Nov 29, 2016
@leastprivilege leastprivilege merged commit f24cb9e into dev Nov 29, 2016
@leastprivilege
Copy link
Member

epic!

@brockallen brockallen deleted the config branch November 30, 2016 13:58
@felschr
Copy link

felschr commented Jan 4, 2017

What's the reason for the removal of IncludeAllClaimsForUser?

@brockallen
Copy link
Member Author

It allowed people to be sloppy. If you want the same semantics, it's easy enough to do that logic in a custom IProfileService.

@lock
Copy link

lock bot commented Jan 11, 2020

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Jan 11, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants