Skip to content
This repository was archived by the owner on Oct 17, 2018. It is now read-only.

Should remove ConfigureDataProtection (make it overload of AddDataProtection) #100

Closed
HaoK opened this issue Nov 2, 2015 · 3 comments
Closed

Should remove ConfigureDataProtection (make it overload of AddDataProtection) #100

HaoK opened this issue Nov 2, 2015 · 3 comments
Assignees
@HaoK
Labels
3 - Done bug
Milestone
1.0.0-rc2

Comments

@HaoK
Copy link
Member

HaoK commented Nov 2, 2015

No description provided.

@muratg
Copy link

muratg commented Nov 12, 2015

Looking at these two methods, this kind of makes sense to me. @GrabYourPitchforks What do you think?

@GrabYourPitchforks
Copy link
Contributor

I'm impartial. Just follow whatever pattern is already established by the other framework components. Also make sure that you don't regress the ability for the developer to configure the system if a separate component calls Add on his behalf.

@HaoK HaoK added this to the 1.0.0-rc2 milestone Nov 17, 2015
@HaoK HaoK self-assigned this Nov 17, 2015
@HaoK HaoK added the bug label Nov 17, 2015
@HaoK
Copy link
Member Author

HaoK commented Nov 17, 2015

0b808ca

@HaoK HaoK closed this as completed Nov 17, 2015
@HaoK HaoK added the 3 - Done label Nov 17, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@HaoK HaoK

Labels
3 - Done bug
Projects
None yet
Milestone
1.0.0-rc2
Development

No branches or pull requests
3 participants
@GrabYourPitchforks @muratg @HaoK