Skip to content

Commit 2e98db2

Browse files
committed
fix account term date processing
test plan: * visit "/accounts/:account_id/terms" * term date fields should accept various date formats as detailed in the ticket (e.g. "MM/DD/YYYY") closes #CNVS-5177 Change-Id: I5ab41502b5b1598cb2d1dee4f485b2e966e87af3 Reviewed-on: https://gerrit.instructure.com/24245 Tested-by: Jenkins <jenkins@instructure.com> QA-Review: August Thornton <august@instructure.com> Reviewed-by: Jeremy Stanley <jeremy@instructure.com> Product-Review: Jeremy Stanley <jeremy@instructure.com>
1 parent 8476de2 commit 2e98db2

1 file changed

Lines changed: 5 additions & 0 deletions

File tree

app/views/terms/index.html.erb

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -102,6 +102,11 @@ $(document).ready(function() {
102102
$(".enrollment_term_form").formSubmit({
103103
processData: function(data) {
104104
var permissions = $(this).parents("tr").find(".permissions").getFormData({object_name: 'enrollment_term'});
105+
$.each(permissions, function(key, val) {
106+
if (key.match(/\[(start)|(end)_at\]/)) {
107+
permissions[key] = $.datetime.process(val);
108+
}
109+
});
105110
return $.extend(permissions, data);
106111
},
107112
beforeSubmit: function(data) {

0 commit comments

Comments
 (0)