Skip to content

Conversation

@agibson-godaddy
Copy link
Contributor

@agibson-godaddy agibson-godaddy commented Nov 25, 2024

Summary

Closes #724
Release: #726

This ensures we only add the "Capture Charge" dropdown item to the orders page and not other pages.

Details

Additional details to expand on the summary, if needed

UI Changes

Before

"Capture Charge" is visible on Posts > All Posts page.

Screenshot 2024-11-25 134516

After

"Capture Charge" is not visible on Posts > All Posts page.

image

But it is still visible on WooCommerce > Orders page.

image

QA

Setup

  1. Install and activate the AuthorizeNet plugin, using its main branch.

Steps

  1. In wp-admin, go to Posts > All Posts and click the bulk action dropdown.
    • You see a "Capture Charge" option!
  2. Switch the AuthorizeNet plugin to use this branch https://github.com/gdcorp-partners/woocommerce-gateway-authorize-net-cim/pull/108 (run composer install again)
  3. In wp-admin, go to Posts > All Posts and click the bulk action dropdown.
    • "Capture Charge" is no longer available.
  4. Go to WooCommerce > Orders and click the bulk action dropdown.
    • "Capture Charge" is visible.

Before merge

  • I have confirmed these changes in each supported minor WooCommerce version

@agibson-godaddy agibson-godaddy self-assigned this Nov 25, 2024
@agibson-godaddy agibson-godaddy changed the base branch from master to release/5.15.2 November 25, 2024 14:06
@agibson-godaddy agibson-godaddy marked this pull request as ready for review November 25, 2024 14:17
@agibson-godaddy agibson-godaddy changed the title Only render on orders page Only render "Capture Charge" bulk action on orders page Nov 25, 2024
@nmolham-godaddy nmolham-godaddy merged commit 3e581b7 into release/5.15.2 Nov 26, 2024
5 checks passed
@nmolham-godaddy nmolham-godaddy deleted the fix/bulk-charge-action branch November 26, 2024 02:59
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.

Capture Charge bulk action option is being added to other list tables

3 participants