Skip to content

Referral#132

Merged
jchen293 merged 3 commits intomasterfrom
referral
May 19, 2022
Merged

Referral#132
jchen293 merged 3 commits intomasterfrom
referral

Conversation

@jchen293
Copy link
Copy Markdown
Contributor

@jchen293 jchen293 commented May 19, 2022 •
edited
Loading

Description

Adds beta Referral class for White Label API with these new functions: create(), updateEmail(), all(), and addCreditCard()
Adds BaseUser class that Referral and User classes can extend from
Adds CreditCard class for addCreditCard() response type

Testing

E2E test
Screen Shot 2022-05-19 at 12 43 32 PM

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)

@jchen293 jchen293 requested review from Justintime50 and nwithan8 May 19, 2022 17:30
@jchen293 jchen293 marked this pull request as ready for review May 19, 2022 17:30
nwithan8
nwithan8 previously approved these changes May 19, 2022
View reviewed changes
@jchen293 jchen293 requested a review from nwithan8 May 19, 2022 19:47
Justintime50
Justintime50 requested changes May 19, 2022
View reviewed changes
Comment thread src/main/java/com/easypost/model/BaseUser.java
@jchen293 jchen293 requested a review from Justintime50 May 19, 2022 20:36
Justintime50
Justintime50 requested changes May 19, 2022
View reviewed changes
Comment thread src/main/java/com/easypost/model/User.java
Comment thread src/main/java/com/easypost/model/beta/Referral.java Outdated
Comment thread src/main/java/com/easypost/model/beta/Referral.java Outdated
Comment thread src/test/java/com/easypost/Fixture.java Outdated
@jchen293 jchen293 merged commit 6599fe0 into master May 19, 2022
@jchen293 jchen293 deleted the referral branch May 19, 2022 21:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Justintime50 Justintime50 Justintime50 approved these changes

+1 more reviewer

@nwithan8 nwithan8 nwithan8 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@jchen293 @nwithan8 @Justintime50