Skip to content

Update dateITA.js #1144

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 2 commits into from
Closed

Update dateITA.js #1144

wants to merge 2 commits into from

Conversation

lboullo0
Copy link
Contributor

@lboullo0 lboullo0 commented Jun 4, 2014

Fixed Bug with dateITA and daylight saving time. Fix #264

Update dateITA.js
cf61ddd 
Fix to  Bug with dateITA and daylight saving time jquery-validation#264
@jzaefferer
Copy link
Collaborator

Thanks. Could you add a unit test for this change?

Update dateITA.js
cb0a852 
Replace == by === to correct build.
@lboullo0
Copy link
Contributor Author

lboullo0 commented Jun 4, 2014

Thank you jzaefferer. I would like to add a unit test, but I don't know how to set an specific timezone.
Another question, I supposed to add the test to test/methods.js ?

@jzaefferer
Copy link
Collaborator

Yes, test/methods.js is the right place. I'm not sure how to deal with the timezone though.

@jzaefferer jzaefferer closed this in a9ba165 Jul 1, 2014
@jzaefferer
Copy link
Collaborator

Figured its better to fix this year-old bug then not due to missing tests. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Bug with dateITA and daylight saving time
2 participants
@lboullo0 @jzaefferer