Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cleanup Identity #109

Merged
merged 9 commits into from
Feb 9, 2022
Merged

Cleanup Identity #109

merged 9 commits into from
Feb 9, 2022

Conversation

fretje
Copy link
Contributor

@fretje fretje commented Feb 6, 2022

  • copy the whole shared project over verbatim from FSH.WebApi
  • rename authentication namespace to auth
  • update changes from api
  • add musthavepermission and use it everywhere

This one goes together with fullstackhero/dotnet-starter-kit#443

* copy the whole shared project over verbatim from FSH.WebApi
* rename authentication namespace to auth
* update changes from api
* add musthavepermission and use it everywhere
@274188A
Copy link
Contributor

274188A commented Feb 7, 2022

is this with a view to making it easier to share between the 2 solutions?

@fretje
Copy link
Contributor Author

fretje commented Feb 7, 2022

is this with a view to making it easier to share between the 2 solutions?

I'm not sure I understand your question.

It's mainly to make things more consistent overall. And the permissions got an overhaul that should make them easier to manage.

@iammukeshm iammukeshm merged commit 8e424e4 into fullstackhero:main Feb 9, 2022
@fretje fretje deleted the Identity branch February 9, 2022 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants