-
Notifications
You must be signed in to change notification settings - Fork 6
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
[Feature] User Creation and Login #16
base: develop
Are you sure you want to change the base?
Conversation
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.
Ready to merge!
Please specify in the readme.MD how another developer can run your project |
And.. How can I test the feature that you added? |
@lubegamark, Please review |
@Dru94 Very Good work so far. I changed the base branch to |
@@ -0,0 +1,3 @@ | |||
from django.test import TestCase | |||
|
|||
# Create your tests here. |
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.
@Dru94 could you add a few tests
from django.contrib.auth.views import LoginView,LogoutView, PasswordResetView, PasswordResetDoneView, \ | ||
PasswordResetConfirmView, PasswordResetCompleteView | ||
|
||
urlpatterns=[ |
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.
@Dru94 what do you think about using social login instead, I'm thinking maybe GitHub and Google. I suggest you use Django Allauth but you can suggest any other package that does the same thing
return render(request, 'base.html') | ||
|
||
#registration view | ||
def reg(request): |
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.
@Dru94 use proper function names, reg
should be register
, I know there's a comment that says registration view but code should be self documenting and require very little comments
Login and registration feature