Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Validate date/time settings when typed in directly. #403

Merged
merged 1 commit into from
Jul 22, 2013

Conversation

peter-fogg
Copy link
Contributor

The changeTime event isn't fired when the user types in the field, but
only when clicking on the time in the dropdown. I'd consider this a
timepicker bug, but for now we can just listen to the user typing in
the field and update the value (and thus validate) like we do with
other field types.

The changeTime event isn't fired when the user types in the field, but
only when clicking on the time in the dropdown. I'd consider this a
timepicker bug, but for now we can just listen to the user typing in
the field and update the value (and thus validate) like we do with
other field types.
@peter-fogg
Copy link
Contributor Author

Presumably nobody's going to get to this until next week, but @cahrens or @singingwolfboy could review.

@chrisndodge
Copy link
Contributor

Looks like a trivial change so it'd be good IMO to get this completed today before the RC is made. Can one other JS person at the company look at it? Maybe @talbs?

@talbs
Copy link
Contributor

talbs commented Jul 15, 2013

Thanks for the heads up, @chrisndodge. I took a spin through the branch locally and things seemed to work just as they currently are on stage. @peter-fogg, is there supposed to be any visible functionality change?

@peter-fogg
Copy link
Contributor Author

@talbs Not a very big one, but it's there -- try setting the course start date to 01:00, and the end date to 02:00 of the same day. This should validate. Then directly type 00:00 into the end date time, and you should see no validation error. This PR causes validation when you type, so setting to midnight will end up showing the error.

@cahrens
Copy link

cahrens commented Jul 22, 2013

This looks fine to me, but run it quickly by @dmitchell. I think he originally was listening to more events, but backed off because methods were getting called too frequently?

I checked out the branch and tested it. Assuming @dmitchell is good, I am 👍

@dmitchell
Copy link
Contributor

👍

peter-fogg pushed a commit that referenced this pull request Jul 22, 2013
Validate date/time settings when typed in directly.
@peter-fogg peter-fogg merged commit c4a21fc into master Jul 22, 2013
@peter-fogg peter-fogg deleted the peter-fogg/fix-stud-421 branch July 22, 2013 15:41
chrisrossi pushed a commit to jazkarta/edx-platform that referenced this pull request Mar 31, 2014
adding maya to supported distro list for create-dev-env.sh
diegomillan pushed a commit to eduNEXT/edx-platform that referenced this pull request Sep 14, 2016
* sig/sig/timestamp:
  Display correct timezone on timestamp for question submission
yoann-mroz pushed a commit to weuplearning/edx-platform that referenced this pull request Nov 30, 2020
MoisesGSalas pushed a commit to eduNEXT/edx-platform that referenced this pull request Jan 12, 2022
…stom-html-templates-in-studio

Pedrovgp/se4781 add custom html templates in studio
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants