Merge pull request #7879 from edx/will/restore-remember-me
Add remember me checkbox back to the combined login/registration page
This commit is contained in:
@@ -220,6 +220,11 @@ $sm-btn-linkedin: #0077b5;
|
||||
font-weight: 500;
|
||||
}
|
||||
|
||||
#login-remember {
|
||||
line-height: 0;
|
||||
height: auto;
|
||||
}
|
||||
|
||||
label {
|
||||
display: block;
|
||||
margin: 0 0 6px 0;
|
||||
@@ -239,6 +244,10 @@ $sm-btn-linkedin: #0077b5;
|
||||
vertical-align: top;
|
||||
}
|
||||
|
||||
&[for="login-remember"] {
|
||||
display: inline-block;
|
||||
}
|
||||
|
||||
a {
|
||||
font-family: $sans-serif;
|
||||
}
|
||||
|
||||
@@ -646,7 +646,18 @@ class LoginSessionViewTest(ApiTestCase):
|
||||
"max_length": PASSWORD_MAX_LENGTH
|
||||
},
|
||||
"errorMessages": {},
|
||||
}
|
||||
},
|
||||
{
|
||||
"name": "remember",
|
||||
"defaultValue": False,
|
||||
"type": "checkbox",
|
||||
"required": False,
|
||||
"label": "Remember me",
|
||||
"placeholder": "",
|
||||
"instructions": "",
|
||||
"restrictions": {},
|
||||
"errorMessages": {},
|
||||
},
|
||||
])
|
||||
|
||||
def test_login(self):
|
||||
|
||||
@@ -102,6 +102,14 @@ class LoginSessionView(APIView):
|
||||
}
|
||||
)
|
||||
|
||||
form_desc.add_field(
|
||||
"remember",
|
||||
field_type="checkbox",
|
||||
label=_("Remember me"),
|
||||
default=False,
|
||||
required=False,
|
||||
)
|
||||
|
||||
return HttpResponse(form_desc.to_json(), content_type="application/json")
|
||||
|
||||
@method_decorator(require_post_params(["email", "password"]))
|
||||
|
||||
Reference in New Issue
Block a user