Skip to content

Conversation

@htdat
Copy link
Member

@htdat htdat commented Jun 11, 2021

Fix #662


Description

This is pretty straightforward as all research has been done in #662

Steps to Test

  1. Apply this PR.
  2. Apply the diff in the Note section for edit_flow.php to avoid caching issue for JS files Fix jQuery warnings as WordPress upgraded it from 1.12.4-wp to 3.5.1 #649
  3. Visit wp-admin > Posts

✅ if we can use Quick Edit and change custom status without any issue.

@htdat htdat requested a review from mikeyarce June 11, 2021 09:20
Copy link
Contributor

@mikeyarce mikeyarce left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this, @htdat !

I tested this out and Quick Edit works as expected. I'm curious as to why this was needed originally... My guess is it had something to do with what WordPress did maybe 10 years ago but is no longer the case.

Either way - Quick Edit custom statuses are loaded correctly and I'm able to select them, save them, change them, etc.
👍🏽

@htdat htdat merged commit 2d09ff9 into master Jun 14, 2021
@htdat htdat deleted the fix/662 branch June 14, 2021 04:02
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.

jQuery - redundant code in modules/custom-status/lib/custom-status.js

3 participants