[Feature][Do not merge] Payouts and Payments#5687
Draft
daledupreez wants to merge 2 commits into
Draft
Conversation
* Add simple admin stripe payment intents REST endpoint * Remove status param and add ending_before param * Rename REST stripe payment intents controller class and add starting_after, ending_before params validation * Add class file to classmap and add controller routes initialisation * Add class file to classmap and add controller routes initialisation * Add return tye to WC_Stripe_REST_Payment_Intents_Controller::validate_payment_id_param * Add endpoint basic unit testing * Add remaining rest search params * Add remaining rest search params * Add remaining rest search params * Add remaining rest search params * Add remaining rest search params * Add remaining rest search params * Add remaining rest search params * Add class description * Fix test method name * Add WC_Stripe_REST_Payment_Intents_Controller::sanitize_created_field * Add WC_Stripe_REST_Payment_Intents_Controller::sanitize_created_field * Remove direct access to WC_Stripe_REST_Payment_Intents_Controller:: * Apply suggested changes * Apply suggested changes * Apply suggested changes * Apply suggested changes * Apply suggested changes * Fix unit test method descriptions * Fix typo * Fix param type in WC_Stripe_REST_Payment_Intents_Controller::build_http_query_array_from_request * Remove 'pre_http_request hook on teardown * Remove controller try - catch block * Add bsic response filtering class * Add basic response filtering class * Add basic response filtering class * Add basic response filtering class * Add basic response filtering class * Add basic response filtering class * Add basic response filtering class * Add basic response filtering class * Refactor WC_Stripe_REST_Response_Filter class * Formatting * Add WC_Stripe_REST_Response_Filter to composer.json classmap * Fix tests * Fix tests * Fix tests * Fix tests * Add changelog entry * Add entry to readme.txt --------- Co-authored-by: Adrian Dobrescu <adobrescu@EN1310580.endava.net>
Contributor
Test the buildIf you are a logged-in Automattic staff member: 🚀 Launch a Jurassic Ninja site with this branch and sample products 🚀 Otherwise, use the following manual flow: 🐢 Navigate to jurassic.ninja 🐢
Build info:
Note: the build is updated when a new commit is pushed to this PR. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changes proposed in this Pull Request:
This PR makes it easier to see what changes are currently included in the
feature/payouts-and-paymentsfeature branch.Testing instructions
Changelog entry
Changelog Entry Comment
Comment
Post merge