Skip to content
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

Update existence checks in newspack_blocks_format_categories #1596

Merged
merged 6 commits into from
Nov 16, 2023

Conversation

DustyReagan
Copy link
Member

@DustyReagan DustyReagan commented Nov 15, 2023

All Submissions:

Changes proposed in this Pull Request:

This PR updates existence checks to prevent Attempt to read property "name"|"term_id" on bool warnings.

Closes p1700077484817269/1700025334.541069-slack-CKZHG0QCR

How to test the changes in this Pull Request:

Follow the testing instructions on this PR to check for regressions.

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 ran tests with your changes locally?

@DustyReagan DustyReagan requested a review from a team as a code owner November 15, 2023 21:43
@DustyReagan DustyReagan changed the title Update existence checks Update existence checks in newspack_blocks_format_categories Nov 15, 2023
@adekbadek adekbadek merged commit 55dbf42 into master Nov 16, 2023
7 checks passed
@adekbadek adekbadek deleted the update/newspack_blocks_format_categories branch November 16, 2023 07:53
@adekbadek
Copy link
Member

Thanks @DustyReagan !

matticbot pushed a commit that referenced this pull request Nov 16, 2023
# [2.2.0-alpha.1](v2.1.0...v2.2.0-alpha.1) (2023-11-16)

### Bug Fixes

* add checks for Newspack plugin ([#1579](#1579)) ([b516a6f](b516a6f))
* **homepage-posts:** terms handling in category list ([#1596](#1596)) ([55dbf42](55dbf42))
* **modal-checkout:** place order button width ([b48efcb](b48efcb)), closes [#1572](#1572)

### Features

* add width control to the Checkout block ([#1583](#1583)) ([3f75c1e](3f75c1e))
* **cwv:** inline HPB styles in a style tag ([#1548](#1548)) ([dea8d93](dea8d93))
* **modal-checkout:** display item name on success ([a3a03df](a3a03df))
* update subscribe patterns to use Newspack block ([#1580](#1580)) ([70088e2](70088e2))
@matticbot
Copy link
Contributor

🎉 This PR is included in version 2.2.0-alpha.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

matticbot pushed a commit that referenced this pull request Nov 27, 2023
# [2.2.0](v2.1.0...v2.2.0) (2023-11-27)

### Bug Fixes

* add checks for Newspack plugin ([#1579](#1579)) ([b516a6f](b516a6f))
* **homepage-posts:** terms handling in category list ([#1596](#1596)) ([55dbf42](55dbf42))
* **modal-checkout:** place order button width ([b48efcb](b48efcb)), closes [#1572](#1572)

### Features

* add width control to the Checkout block ([#1583](#1583)) ([3f75c1e](3f75c1e))
* **cwv:** inline HPB styles in a style tag ([#1548](#1548)) ([dea8d93](dea8d93))
* **modal-checkout:** display item name on success ([a3a03df](a3a03df))
* update subscribe patterns to use Newspack block ([#1580](#1580)) ([70088e2](70088e2))
@matticbot
Copy link
Contributor

🎉 This PR is included in version 2.2.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants