Skip to content

Conversation

@masteradhoc
Copy link
Contributor

All Submissions:

Changes proposed in this Pull Request:

Closes #177 .

How to test the changes in this Pull Request:

  1. create new products
  2. see if tax_status is randomly generated

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?

Changelog entry

Dynamic assign tax_status for simple and variable products

Enter a summary of all changes on this Pull Request. This will appear in the changelog if accepted.

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.

@masteradhoc
Copy link
Contributor Author

@layoutd can you check this PR as well and let me know about potential change requests?

Copy link
Contributor

@layoutd layoutd left a comment

Choose a reason for hiding this comment

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

LGTM

@layoutd layoutd merged commit 18c2e99 into woocommerce:trunk Oct 30, 2025
@masteradhoc masteradhoc deleted the patch-5 branch October 30, 2025 12:12
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.

[Enhancement]: Dynamic tax_status

2 participants