Skip to content

Conversation

@justasdobrovolskis
Copy link
Contributor

@justasdobrovolskis justasdobrovolskis commented Aug 29, 2022

All Submissions:

Changes proposed in this Pull Request:

Closes # .

How to test the changes in this Pull Request:

  1. Enable tracking via WooCommerce ▸ Settings ▸ Advanced ▸ WooCommerce.com.
  2. Optionally, install and activate Query Monitor (makes it easy to see errors—but you can use some other tool or method if you prefer).
    • If you do choose to use Query Monitor, you will probably also want to ensure you can see your admin toolbar while in the block editor.
    • To do this, configure the editor so it is not operating in full-screen mode:

blocks-full-screen

  1. Try adding a new post via Posts ▸ Add New.
    • With this change, everything should be fine!
    • Without this change, you may observe a warning about Undefined array key "post_type":

undefined-array-key

Other information:

  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your changes, as applicable?
  • Have you successfully run tests with your changes locally?
  • Have you created a changelog file for each project being changed, ie pnpm changelog add --filter=<project>?

FOR PR REVIEWER ONLY:

  • I have reviewed that everything is sanitized/escaped appropriately for any SQL or XSS injection possibilities. I made sure Linting is not ignored or disabled.

@github-actions github-actions bot added the plugin: woocommerce Issues related to the WooCommerce Core plugin. label Aug 29, 2022
Copy link
Member

@barryhughes barryhughes left a comment

Choose a reason for hiding this comment

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

Thanks, looks good to me!

  • Changelog added
  • Testing instructions added
  • Will circle back and merge once checks pass

(+Apologies for the delay in reviewing.)

@barryhughes barryhughes merged commit d4f2212 into woocommerce:trunk Nov 30, 2022
@github-actions github-actions bot added this to the 7.3.0 milestone Nov 30, 2022
psealock added a commit to psealock/woocommerce that referenced this pull request Dec 1, 2022
* Use workflow dispatch that now works with GITHUB_TOKEN

* check if post_type exists (woocommerce#34501)

* check if post_type exists

* Changelog.

Co-authored-by: barryhughes <[email protected]>

* Fix Protected Directory Detection and Admin Notice (woocommerce#33544)

* Fix Protected Directory Admin Notice

- make notice actually close when clicking "Dismiss"
- make unprotected directory detection smarter: not every /woocommerce_uploads/ page with content is a directory listing

* Unwind the change made to method `is_uploads_directory_protected()`.

Discussion here https://github.com/woocommerce/woocommerce/pull/33544/files#r910253861 relates. In essence, we felt it better to have more warnings prompting the user to evaluate if the directory is protected, than fewer warnings with an increased change of not flagging actual problems.

* Add changelog.

Co-authored-by: barryhughes <[email protected]>

* Update wc-account-functions.php (woocommerce#32026)

* Update wc-account-functions.php

Correct order of single/plural translation strings to prevent confusion with translations and translators.

* Changelog.

Co-authored-by: barryhughes <[email protected]>

* Set editor mode on initialization to prevent initial text editor focus (woocommerce#35701)

* Set editor mode on initialization to prevent initial text editor focus

* Add changelog entry

* Check for editor set method to prevent breaks in tests with older versions of block store

* Update argument name for initialization

* Use workflow dispatch that now works with GITHUB_TOKEN (#6)

* Fix indentation of code freeze yml file

* Dev/use workflow dispatch (#8)

* Fix indentation of code freeze yml file

* Use context to get repo and owner.

* Send the required inputs to dispatched workflow.

Co-authored-by: Sam Seay <[email protected]>
Co-authored-by: justasdobrovolskis <[email protected]>
Co-authored-by: barryhughes <[email protected]>
Co-authored-by: Andrew <[email protected]>
Co-authored-by: Philip <[email protected]>
Co-authored-by: Joshua T Flowers <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

plugin: woocommerce Issues related to the WooCommerce Core plugin. type: community contribution

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants