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

Asp.net Identity update #37

Closed
rayrfan opened this issue Sep 29, 2017 · 0 comments
Closed

Asp.net Identity update #37

rayrfan opened this issue Sep 29, 2017 · 0 comments
Assignees
Labels
Auth authentication and authorization DB Change db schema or sql upgrade script changes
Milestone

Comments

@rayrfan
Copy link
Owner

rayrfan commented Sep 29, 2017

Use int PK for all tables (done)

By default the aspnet tables use string as the PK. Here are two ref on how to change that.

Rename table names (done)

  • Rename aspnet table to "Fan_" and use singular table name instead of plural such as
    AspNetRoles -> Fan_Roles
  • Add "Blog_" prefix to other tables.

Decide on name and email when sign up and login (not right now)

I want to eventually support login with either a username or email but not right now due to lack of time, once you add a Username field, you have to

  • Validate username (a-z,0-9 that sort thing)
  • Verify if incoming username is email or not, if it's email search use by it and get the real username

So for right now just use email and fill the UserName column with email. In future I can use username login on other apps.

Add default "Admin" Role during blog setup (done)

Should I add the following properties to Role table

  • IsSystemRole
  • Description

This is done in BlogController Setup post.
https://stackoverflow.com/questions/42471866/how-to-create-roles-in-asp-net-core-and-assign-them-to-users

Decide whether to keep Migrations as part of my repo (done)

Based on my research there is no easy way to support multiple database vendors with migration.

@rayrfan rayrfan added the BLL label Sep 29, 2017
@rayrfan rayrfan added this to the 1.0.0-alpha milestone Sep 29, 2017
@rayrfan rayrfan self-assigned this Sep 29, 2017
@rayrfan rayrfan modified the milestones: 1.0.0-alpha, 1.0.0-beta Oct 5, 2017
@rayrfan rayrfan added the DB Change db schema or sql upgrade script changes label Oct 9, 2017
rayrfan pushed a commit that referenced this issue Oct 13, 2017
rayrfan pushed a commit that referenced this issue Oct 13, 2017
… setup, switched to use UserId instead of UserName on Post.
@rayrfan rayrfan closed this as completed Oct 16, 2017
@rayrfan rayrfan added Auth authentication and authorization and removed Identity labels Aug 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Auth authentication and authorization DB Change db schema or sql upgrade script changes
Projects
No open projects
Development

No branches or pull requests

1 participant