Automattic / WP-Job-Manager

Manage job listings from the WordPress admin panel, and allow users to post jobs directly to your site.
https://wpjobmanager.com
GNU General Public License v3.0
895 stars 366 forks source link

Set the default dateFormat if it’s empty #2779

Closed mikeyarce closed 3 months ago

mikeyarce commented 4 months ago

Fixes https://github.com/Automattic/WP-Job-Manager/issues/2739

Changes Proposed in this Pull Request

Testing Instructions

Release Notes


Plugin build for 5b318b12806dbc22f58216ca0f4fe37f9b16b4b5
📦 Download plugin zip
▶️ Open in playground
mikeyarce commented 4 months ago

Thanks @thedebian - my initial PR was addressing the problem when jQuery UI was missing the format, but now I've expanded that to add a fallback for when the actual get_option( 'date_format' ) is missing in the rest of the plugin 😄

ibiza69 commented 3 months ago

Hello, we updated the plugin with the patch provided on github and it completely broke our website, both the front and backend, without the possibility of access to both. We have had to download the original plugin (not the one patched at github), version 2.2.2 from WordPress and upload it via FTP, overwriting the files, in order to solve the problem.

mikeyarce commented 3 months ago

Hi @ibiza69 - did you just download the source or did you also run the build steps? The safest bet would be to wait till we have a release and test that!

ibiza69 commented 3 months ago

Hi @mikeyarce, we downloaded thru github the fix and uploaded the files thru FTP and that crashed our site.

We shall wait untill plugin update at wordpress :) Thank you.