Removed django-registration-redux#106
Removed django-registration-redux#106anubhakushwaha merged 2 commits intoOpenSourceHelpCommunity:developfrom vaibhavsingh97:develop
Conversation
vaibhavsingh97
requested review from
anubhakushwaha,
nikhita and
tapaswenipathak
| ('Ibrahim Jarif','jarifibrahim@gmail.com'), | ||
| ('Tapasweni Pathak', 'tapaswenipathak@gmail.com'), | ||
| ('Nikhita Raghunath', 'nikitaraghunath@gmail.com'), | ||
| ('Ibrahim Jarif', 'jarifibrahim@gmail.com'), |
There was a problem hiding this comment.
@anubhakushwaha According to PEP 8, we should have spaces. If you want i can remove.
There was a problem hiding this comment.
@vaibhavsingh97 Ooh that's interesting, we would love to learn more,Is this the place? https://www.python.org/dev/peps/pep-0008/#tabs-or-spaces
There was a problem hiding this comment.
oshc/oshc/settings.py
Outdated
| # and are trying to access pages requiring authentication | ||
| LOGIN_URL = '/accounts/login/' | ||
| LOGOUT_REDIRECT_URL = 'home' | ||
|
|
anubhakushwaha
left a comment
anubhakushwaha
left a comment
There was a problem hiding this comment.
Just the minor changes, else LGTM
anubhakushwaha
left a comment
anubhakushwaha
left a comment
There was a problem hiding this comment.
Remove unit tests for now
|
@anubhakushwaha I had made suggested change. Please merge |
|
@vaibhavsingh97 Just mention the reason for the decrease in coverage for everyone and it's ready to be merged. |
|
@anubhakushwaha @tapasweni-pathak @nikhita Please find the detailed report 📓 Reason for decrease in coverageBefore removal of test case:
Statement covered: 102 After removal of test case:
Statement covered: 51 |
|
Merging, keeping the two commits separate just as a reminder that we have removed the test cases as well. |
3 participants
Status:
Which issue does this PR fix?:
fixes #104
Brief description of what this PR does:
This PR is for removing django-registration-redux model.