-
Notifications
You must be signed in to change notification settings - Fork 368
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
Fix for the hidden and displayed datepicker fields not holding the cu… #2766
Fix for the hidden and displayed datepicker fields not holding the cu… #2766
Conversation
…rrent values for _job_expires in datepicker.js.
…nverted to using camelCase.
1f469e5
to
57e1e2d
Compare
…n the if branch in the added function 'dateFormatFunction'.
656f093
to
5957663
Compare
…ring the verbose Date output.
…stency. Moved the code for applying the value fixes to the hidden and target fields at the end of the 'initializeDatepicker' function's code block.
d5acc05
to
ebe347c
Compare
Not sure how important it was to ensure that the names I added were camelCase (as I see a bit of mixing of naming styles: snake_case and "$snake_case" in some places. However, I won't touch any of that code as that would be out of scope for the purpose of this PR). The need to add the The fix to the field values now occurs at the end of the Note that at the end of I've tested my changes on my client's WP install to ensure this fix is still working. (also note the force pushes were just to amend some typos in my commit messages) |
…sting-Expiry-Date-Fix-Issue-2739
Closing this as this other PR fixes it #2779 |
Fix for the hidden and displayed datepicker fields not holding the current values for _job_expires in datepicker.js.
Fixes #2739
Changes Proposed in this Pull Request
Testing Instructions
Release Notes