-
-
Notifications
You must be signed in to change notification settings - Fork 764
convert use of URI to string in models #620
Conversation
@@ -73,17 +73,17 @@ public class Client | |||
/// <summary> | |||
/// Specifies allowed URIs to return tokens or authorization codes to | |||
/// </summary> | |||
public List<Uri> RedirectUris { get; set; } | |||
public ICollection<string> RedirectUris { get; set; } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't really like ICollection - since this Model is used to define clients in code manually - List<T>
plays much nicer with intellisense.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lern2linq :P
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it is not about linq - it is about being able to press tab in VS (and not getting #if)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't follow (or agree), but that's ok -- I reverted it back.
looks good now |
I'll do this merge friday or over the weekend -- it'll require many updates in the samples. |
convert use of URI to string in models
for discussion