Closed frosso closed 3 days ago
9783
or branch name feat/add-gateway-label-filter-override
in your-test.site/wp-admin/admin.php?page=jetpack-beta&plugin=woocommerce-payments :rocket: Launch a JN site with this branch :rocket:
:information_source: Install this Tampermonkey script to get more options.
Build info:
Note: the build is updated when a new commit is pushed to this PR.
Size Change: 0 B
Total Size: 1.33 MB
@pierorocca for visibility on the incompatibilities - in Heisenberg we'll need to revisit the feasibility of the current approach (CC @FangedParakeet ) I added a topic for our roundtable.
Fixes #
Changes proposed in this Pull Request
Discussed here: https://github.com/Automattic/woocommerce-payments/pull/9647#issuecomment-2490796584
The "Test mode" label that we originally introduced in 8.4.0 can have some incompatibilities with themes or existing pages. We already fixed some incompatibility with the "change payment method" page for subscriptions. But I think a bigger revision might be needed. Adding these styles via the
get_title()
method on the main gateway seems to be problematic for some themes ( p1731488040924719/1731451895.464649-slack-CU6SYV31A | https://github.com/Automattic/woocommerce-payments/pull/9647#issuecomment-2490956335 ). We now found that it could also affect email templates: https://github.com/Automattic/woocommerce-payments/issues/9779For now, we can add a filter for merchants to tap into.
I'm also tempted to add an
is_ajax()
check on theget_title()
method, just to fix the email address. But for now, let's move forward with at least having a filter.This filter doesn't alter the block checkout, which doesn't suffer of this issue.
Testing instructions
npm run changelog
to add a changelog file, choosepatch
to leave it empty if the change is not significant. You can add multiple changelog files in one PR by running this command a few times.Post merge