Refactor role claim parsing for issue #595#601
Conversation
|
ziagham
left a comment
There was a problem hiding this comment.
The changes look good overall. Thanks.
However, the unit tests should have been placed in a separate project named FlowSynx.UnitTests. Although that project doesn’t currently exist and needs to be created, it’s not a major issue. I’ll create a separate task to add a new FlowSynx.UnitTests project and move all related tests there.
|
Hi @ziagham, thanks for appreciating my work! |
|
@mohin-io |
|
That's awesome, @ziagham ! Thank you for the trust and the opportunity. |



Summary
Implementation Notes
Testing
Closes #595