Skip to content

Conversation

jchen293
Copy link
Contributor

@jchen293 jchen293 commented Dec 15, 2022

Description

Add new beta billing functionality for ReferralCustomer users

  • addPaymentMethod can add a pre-existing Stripe bank account or credit card to your EasyPost account
  • refundByAmount refunds your wallet by a dollar amount
  • refundByPaymentLog refunds your wallet by a PaymentLog ID

Testing

Add new unit tests

Pull Request Type

Please select the option(s) that are relevant to this PR.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Improvement (fixing a typo, updating readme, renaming a variable name, etc)

@Justintime50
Copy link
Member

This is no longer valid since the functionality provided by Stripe is deprecated.

@Justintime50 Justintime50 reopened this Jan 4, 2023
@jchen293 jchen293 force-pushed the stripe_refund branch 2 times, most recently from cadd9b3 to e600c34 Compare January 4, 2023 22:32
@jchen293 jchen293 marked this pull request as ready for review January 4, 2023 22:36
@jchen293 jchen293 requested a review from a team January 4, 2023 22:36
Justintime50
Justintime50 previously approved these changes Jan 5, 2023
@jchen293
Copy link
Contributor Author

jchen293 commented Jan 5, 2023

Force pushed to rebase and changed REFERRAL_USER_PROD_API_KEY to REFERRAL_CUSTOMER_PROD_API_KEY, and refund_amount_subunits to refund_amount

@EasyPost EasyPost deleted a comment from Justintime50 Jan 5, 2023

@Getter
public class BetaPaymentRefund extends EasyPostResource{
private String refundedAmount;
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably, the responses in our internal doc are string and int, looks like I need to add an attribute for refunded_amount_currencys

Justintime50
Justintime50 previously approved these changes Jan 5, 2023
@jchen293 jchen293 requested a review from nwithan8 January 6, 2023 21:22
@jchen293 jchen293 merged commit 1be0099 into master Jan 9, 2023
@jchen293 jchen293 deleted the stripe_refund branch January 9, 2023 22:10
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