Skip to content
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

Add refund function to insurance service #440

Merged
merged 4 commits into from
Apr 8, 2024
Merged

Conversation

jchen293
Copy link
Contributor

@jchen293 jchen293 commented Apr 2, 2024

Description

Add refund function to insurance service

Testing

Add a new unit test

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 a review from a team April 2, 2024 20:32
@jchen293 jchen293 marked this pull request as draft April 2, 2024 20:32
Copy link
Contributor

@nwithan8 nwithan8 left a comment

Choose a reason for hiding this comment

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

See comments

src/services/insurance_service.js Outdated Show resolved Hide resolved
@jchen293 jchen293 force-pushed the insurance_cancellation branch from fb4fd25 to 610a31e Compare April 4, 2024 16:52
@jchen293 jchen293 marked this pull request as ready for review April 4, 2024 16:54
@jchen293 jchen293 requested a review from nwithan8 April 4, 2024 16:54
Justintime50
Justintime50 previously approved these changes Apr 4, 2024
nwithan8
nwithan8 previously approved these changes Apr 4, 2024
@jchen293 jchen293 dismissed stale reviews from nwithan8 and Justintime50 via 1383f80 April 8, 2024 17:56
@jchen293 jchen293 force-pushed the insurance_cancellation branch from 610a31e to 1383f80 Compare April 8, 2024 17:56
@jchen293 jchen293 merged commit 0a05ddf into master Apr 8, 2024
8 checks passed
@jchen293 jchen293 deleted the insurance_cancellation branch April 8, 2024 18:35
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