Skip to content

Conversation

Abhicodes-crypto
Copy link
Contributor

Type of Change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring
  • Dependency updates
  • Documentation
  • CI/CD

Description

Fix refunds , fetch refunds if the insert fails.

Additional Changes

  • This PR modifies the API contract
  • This PR modifies the database schema
  • This PR modifies application configuration/environment variables

Motivation and Context

How did you test it?

CI checks, as CI checks were failing before.

Checklist

  • I formatted the code cargo +nightly fmt --all
  • I addressed lints thrown by cargo clippy
  • I reviewed the submitted code
  • I added unit tests for my changes where possible
  • I added a CHANGELOG entry if applicable

@Abhicodes-crypto Abhicodes-crypto added the A-core Area: Core flows label Oct 25, 2023
@Abhicodes-crypto Abhicodes-crypto self-assigned this Oct 25, 2023
@Abhicodes-crypto Abhicodes-crypto requested a review from a team as a code owner October 25, 2023 09:41
@Abhicodes-crypto Abhicodes-crypto added S-waiting-on-review Status: This PR has been implemented and needs to be reviewed and removed S-waiting-on-review Status: This PR has been implemented and needs to be reviewed labels Oct 25, 2023
@Abhicodes-crypto Abhicodes-crypto marked this pull request as draft October 25, 2023 09:53
@Abhicodes-crypto Abhicodes-crypto marked this pull request as ready for review October 25, 2023 11:26
@Gnanasundari24 Gnanasundari24 added this pull request to the merge queue Oct 25, 2023
Merged via the queue into main with commit 433cdfa Oct 25, 2023
@Gnanasundari24 Gnanasundari24 deleted the fix-refunds branch October 25, 2023 14:02
Abhicodes-crypto added a commit that referenced this pull request Nov 6, 2023
Abhicodes-crypto added a commit that referenced this pull request Nov 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-core Area: Core flows
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants