Skip to content
Projects
Groups
Snippets
Help
This project
Loading...
Sign in / Register
Toggle navigation
E
edx-platform
Overview
Overview
Details
Activity
Cycle Analytics
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Charts
Issues
0
Issues
0
List
Board
Labels
Milestones
Merge Requests
0
Merge Requests
0
CI / CD
CI / CD
Pipelines
Jobs
Schedules
Charts
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Charts
Create a new issue
Jobs
Commits
Issue Boards
Open sidebar
edx
edx-platform
Commits
794e403b
Commit
794e403b
authored
Nov 19, 2015
by
Matt Drayer
Browse files
Options
Browse Files
Download
Plain Diff
Merge pull request #10635 from edx/asadiqbal08/SOL-1416
asadiqbal08/SOL-1416 Certs: Remove Restriction on Date Setting
parents
42c2cc4d
88ae4ce2
Hide whitespace changes
Inline
Side-by-side
Showing
2 changed files
with
0 additions
and
10 deletions
+0
-10
cms/static/js/models/settings/course_details.js
+0
-3
cms/static/js/spec/views/settings/main_spec.js
+0
-7
No files found.
cms/static/js/models/settings/course_details.js
View file @
794e403b
...
...
@@ -35,9 +35,6 @@ var CourseDetails = Backbone.Model.extend({
if
(
newattrs
.
start_date
===
null
)
{
errors
.
start_date
=
gettext
(
"The course must have an assigned start date."
);
}
if
(
this
.
hasChanged
(
"start_date"
)
&&
this
.
get
(
"has_cert_config"
)
===
false
){
errors
.
start_date
=
gettext
(
"The course must have at least one active certificate configuration before it can be started."
);
}
if
(
newattrs
.
start_date
&&
newattrs
.
end_date
&&
newattrs
.
start_date
>=
newattrs
.
end_date
)
{
errors
.
end_date
=
gettext
(
"The course end date must be later than the course start date."
);
}
...
...
cms/static/js/spec/views/settings/main_spec.js
View file @
794e403b
...
...
@@ -72,13 +72,6 @@ define([
);
});
it
(
'Changing course start date without active certificate configuration should result in error'
,
function
()
{
this
.
view
.
$el
.
find
(
'#course-start-date'
)
.
val
(
'10/06/2014'
)
.
trigger
(
'change'
);
expect
(
this
.
view
.
$el
.
find
(
'span.message-error'
).
text
()).
toContain
(
"course must have at least one active certificate configuration"
);
});
it
(
'Selecting a course in pre-requisite drop down should save it as part of course details'
,
function
()
{
var
pre_requisite_courses
=
[
'test/CSS101/2012_T1'
];
var
requests
=
AjaxHelpers
.
requests
(
this
),
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment