Skip to content

Conversation

@peterwilsoncc
Copy link
Collaborator

@peterwilsoncc peterwilsoncc commented Nov 27, 2025

All Submissions:

  • Does your code follow the WooCommerce Sniffs variant of WordPress coding standards?
  • Have you written new tests for your changes, as applicable?
  • Have you successfully run tests with your changes locally?
  • Will this change require new documentation or changes to existing documentation?

Changes proposed in this Pull Request:

Ensures the default gateway titles are displayed if not set in the options.

This fixes an issue in which check-out pages does not show any text for the payment method when the settings page for the gateway has not been visited by the merchant. This can happen if they click the enable button on WooCommerce > Settings > Payments page without clicking the manage button.

Bug on Shortcode checkout Bug on Block checkout
Screenshot 2025-07-30 at 2 07 41 pm Screenshot 2025-07-30 at 2 06 00 pm

Closes https://linear.app/a8c/issue/SQUARE-150/checkout-title-does-not-display-prior-to-managing-and-saving-payment

Steps to test the changes in this Pull Request:

  1. In the database delete the options woocommerce_square_cash_app_pay_settings and woocommerce_square_credit_card_settings.
  2. Log in to the dashboard and visit the WooCommerce > Settings > Payments page
  3. Click the enable button for Square, do not click the manage button.
  4. Click the enable button for Cash App Pay (Square), do not click the manage button.
  5. Repeat the following steps for both the block and shortcode checkouts:
    1. Add a product to the cart
    2. Proceed to checkout
    3. Ensure the default titles for each payment methods is displayed

Changelog entry

Fix - Ensure default titles are displayed for payment methods if not set by the merchant.

@peterwilsoncc peterwilsoncc self-assigned this Dec 2, 2025
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.

3 participants