-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Conversation
…to latest version.
|
@TomasHubelbauer - what do you think about it? :) |
|
Please break the PR into two next time there's this many changes, some of which not reviewable (like |
|
Great tip, I agree. :) |
|
Ah, nevermind, so in this case when I collapsed all the |
| var grants = await _interaction.GetAllUserConsentsAsync(); | ||
|
|
||
| var list = new List<GrantViewModel>(); | ||
| foreach(var grant in grants) |
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.
You're missing a space after foreach if you format code that way.
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.
DONE. :)
|
|
||
| // this enables automatic token cleanup. this is optional. | ||
| options.EnableTokenCleanup = true; | ||
| // options.TokenCleanupInterval = 15; // frequency in seconds to cleanup stale grants. 15 is useful during debugging |
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.
Should this be here if commented out? Maybe #if DEBUG?
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.
DONE. :)
I've added new project with IDS4 that is connected to Asp.Net Core Identity and EF - #30
@xmichaelx - Please take a look. :)
Thank you.