Skip to content

Conversation

@dkoo
Copy link
Contributor

@dkoo dkoo commented Nov 19, 2025

All Submissions:

Changes proposed in this Pull Request:

While testing for #4315, I discovered a bug that causes sites to store and read metering data from the wrong meta key if using Woo Memberships for content gating. This would only affects sites that have previously enabled the experimental content gating UI already, so it shouldn't affect any sites in production.

How to test the changes in this Pull Request:

  1. Make sure NEWSPACK_CONTENT_GATES is not defined in wp-config.php.
  2. With Woo Memberships enabled and with a plan that restricts content by active subscription, publish a content gate with metering enabled for anonymous and logged-in readers.
  3. As a reader, confirm that metering works as expected while both not logged in and logged in.

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?

@dkoo dkoo self-assigned this Nov 19, 2025
@dkoo dkoo requested a review from a team as a code owner November 19, 2025 00:01
@dkoo dkoo added the [Status] Needs Review The issue or pull request needs to be reviewed label Nov 19, 2025
@github-actions github-actions bot added [Status] Approved The pull request has been reviewed and is ready to merge and removed [Status] Needs Review The issue or pull request needs to be reviewed labels Nov 25, 2025
@dkoo dkoo merged commit a190a47 into release Nov 25, 2025
10 checks passed
@dkoo dkoo deleted the fix/metering-meta-key-w-memberships branch November 25, 2025 16:08
matticbot pushed a commit that referenced this pull request Nov 25, 2025
## [6.25.1](v6.25.0...v6.25.1) (2025-11-25)

### Bug Fixes

* **memberships:** use gate ID for metering meta key ([#4316](#4316)) ([a190a47](a190a47))
@matticbot
Copy link
Contributor

🎉 This PR is included in version 6.25.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Labels

released [Status] Approved The pull request has been reviewed and is ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants