-
Notifications
You must be signed in to change notification settings - Fork 10.8k
Revert feature name to add_payment_method #60197
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
📝 WalkthroughWalkthroughThis change standardizes the use of a payment gateway feature constant throughout the WooCommerce codebase. Specifically, it renames the Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~7 minutes Note ⚡️ Unit Test Generation is now available in beta!Learn more here, or try it out under "Finishing Touches" below. 📜 Recent review detailsConfiguration used: .coderabbit.yml 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
✨ Finishing Touches🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
Testing GuidelinesHi , Apart from reviewing the code changes, please make sure to review the testing instructions (Guide) and verify that relevant tests (E2E, Unit, Integration, etc.) have been added or updated as needed. Reminder: PR reviewers are required to document testing performed. This includes:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix looks good to me! Thank you for your contribution 🙌
Glad to contribute. |
@mohsinoffline, thanks for the heads up in #57468 (comment) 🙌 Can you share some extra context on how you discovered this issue (so we know if we need to do anything beyond merging this PR)? |
On Slack @mohsinoffline told me about this issue. |
Gotcha @szepeviktor, and thanks for crafting this PR! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM and CI is green!
* Revert feature name to add_payment_method * Create 60197-revert-feature-name --------- Co-authored-by: Vladimir Reznichenko <kalessil@gmail.com>
Submission Review Guidelines:
Changes proposed in this Pull Request:
#57468 Introduced a breaking change: changed feature name from add_payment_method to add_payment_methods
Many WC plugins use add_payment_method
Inspired by @mohsinoffline