Merge pull request #15169 from edx/saleem-latif/ENT-329-updates
ENT-329: Update account activation message on login page
This commit is contained in:
@@ -174,7 +174,7 @@ class TestActivateAccount(TestCase):
|
||||
# Access activation link, message should say that account has been activated.
|
||||
response = self.client.get(reverse('activate', args=[self.registration.activation_key]), follow=True)
|
||||
self.assertRedirects(response, login_page_url)
|
||||
self.assertContains(response, 'You have activated your account.')
|
||||
self.assertContains(response, 'Success! You have activated your account.')
|
||||
|
||||
# Access activation link again, message should say that account is already active.
|
||||
response = self.client.get(reverse('activate', args=[self.registration.activation_key]), follow=True)
|
||||
|
||||
@@ -2315,10 +2315,21 @@ def activate_account(request, key):
|
||||
else:
|
||||
if not registration.user.is_active:
|
||||
registration.activate()
|
||||
# Add account activation success message for display later
|
||||
# Success message for logged in users.
|
||||
message = _('{html_start}Success{html_end} You have activated your account.')
|
||||
|
||||
if not request.user.is_authenticated():
|
||||
# Success message for logged out users
|
||||
message = _(
|
||||
'{html_start}Success! You have activated your account.{html_end}'
|
||||
'You will now receive email updates and alerts from us related to'
|
||||
' the courses you are enrolled in. Sign In to continue.'
|
||||
)
|
||||
|
||||
# Add message for later use.
|
||||
messages.success(
|
||||
request,
|
||||
HTML(_('{html_start}Success{html_end} You have activated your account.')).format(
|
||||
HTML(message).format(
|
||||
html_start=HTML('<p class="message-title">'),
|
||||
html_end=HTML('</p>'),
|
||||
),
|
||||
|
||||
Reference in New Issue
Block a user