diff --git a/common/djangoapps/student/tests/test_bulk_email_settings.py b/common/djangoapps/student/tests/test_bulk_email_settings.py index 4b4b6df04d..0fd2a62f3b 100644 --- a/common/djangoapps/student/tests/test_bulk_email_settings.py +++ b/common/djangoapps/student/tests/test_bulk_email_settings.py @@ -86,6 +86,7 @@ class TestStudentDashboardEmailView(ModuleStoreTestCase): # Assert that the URL for the email view is not in the response # if this course isn't authorized response = self.client.get(self.url) + print response.content self.assertTrue(self.email_modal_link in response.content) diff --git a/lms/templates/dashboard.html b/lms/templates/dashboard.html index ac53136051..8c3f1749cf 100644 --- a/lms/templates/dashboard.html +++ b/lms/templates/dashboard.html @@ -168,7 +168,6 @@ ${state.provider.NAME} - % if state.has_account:
- ## Translators: clicking on this removes the link between a user's edX account and their account with an external authentication provider (like Google or LinkedIn). - ${_("Unlink")} + ## Translators: clicking on this removes the link between a user's edX account and their account with an external authentication provider (like Google or LinkedIn). + ${_("Unlink")} - % else: +
+ % else: - ## Translators: clicking on this creates a link between a user's edX account and their account with an external authentication provider (like Google or LinkedIn). - ${_("Link")} + ## Translators: clicking on this creates a link between a user's edX account and their account with an external authentication provider (like Google or LinkedIn). + ${_("Link")} % endif - -
% endfor + + % endif diff --git a/lms/templates/dashboard/_dashboard_course_listing.html b/lms/templates/dashboard/_dashboard_course_listing.html index 77baa1e956..3147641c63 100644 --- a/lms/templates/dashboard/_dashboard_course_listing.html +++ b/lms/templates/dashboard/_dashboard_course_listing.html @@ -41,11 +41,9 @@ from student.helpers import ( <% mode_class = '' %> % endif -
+
- <% - course_target = reverse('info', args=[course.id.to_deprecated_string()]) - %> + <% course_target = reverse('info', args=[course.id.to_deprecated_string()]) %> % if show_courseware_link: % if not is_course_blocked: @@ -72,30 +70,31 @@ from student.helpers import ( % endif -
-
-

- % if course.has_ended(): - ${_("Course Completed - {end_date}").format(end_date=course.end_datetime_text("DATE_TIME"))} - % elif course.has_started(): - ${_("Course Started - {start_date}").format(start_date=course.start_datetime_text("DATE_TIME"))} - % elif course.start_date_is_still_default: # Course start date TBD - ${_("Course has not yet started")} - % else: # hasn't started yet - ${_("Course Starts - {start_date}").format(start_date=course.start_datetime_text("DATE_TIME"))} - % endif -

-
-
+
+
+

+ % if course.has_ended(): + ${_("Course Completed - {end_date}").format(end_date=course.end_datetime_text("DATE_TIME"))} + % elif course.has_started(): + ${_("Course Started - {start_date}").format(start_date=course.start_datetime_text("DATE_TIME"))} + % elif course.start_date_is_still_default: # Course start date TBD + ${_("Course has not yet started")} + % else: # hasn't started yet + ${_("Course Starts - {start_date}").format(start_date=course.start_datetime_text("DATE_TIME"))} + % endif +

+
+
+
% if show_courseware_link: % if not is_course_blocked: - ${_('{course_number} {course_name} Home Page').format(course_number=course.number, course_name=course.display_name_with_default) |h} - - % else: + ${_('{course_number} {course_name} Home Page').format(course_number=course.number, course_name=course.display_name_with_default) |h} + + % else: - ${_('{course_number} {course_name} Cover Image').format(course_number=course.number, course_name=course.display_name_with_default) |h} - + ${_('{course_number} {course_name} Cover Image').format(course_number=course.number, course_name=course.display_name_with_default) |h} + % endif % else:
@@ -115,9 +114,9 @@ from student.helpers import (

% if show_courseware_link: % if not is_course_blocked: - ${course.display_name_with_default} + ${course.display_name_with_default} % else: - ${course.display_name_with_default} + ${course.display_name_with_default} % endif % else: ${course.display_name_with_default} @@ -128,13 +127,13 @@ from student.helpers import ( ${course.display_number_with_default | h} % if course.has_ended(): - ${_("Ended - {end_date}").format(end_date=course.end_datetime_text("SHORT_DATE"))} + ${_("Ended - {end_date}").format(end_date=course.end_datetime_text("SHORT_DATE"))} % elif course.has_started(): - ${_("Started - {start_date}").format(start_date=course.start_datetime_text("SHORT_DATE"))} + ${_("Started - {start_date}").format(start_date=course.start_datetime_text("SHORT_DATE"))} % elif course.start_date_is_still_default: # Course start date TBD - ${_("Coming Soon")} + ${_("Coming Soon")} % else: # hasn't started yet - ${_("Starts - {start_date}").format(start_date=course.start_datetime_text("SHORT_DATE"))} + ${_("Starts - {start_date}").format(start_date=course.start_datetime_text("SHORT_DATE"))} % endif

@@ -252,10 +251,6 @@ from student.helpers import ( % endif - - - -
-
- +