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

feat: api routes #16

Merged
merged 2 commits into from
Jul 4, 2024
Merged

feat: api routes #16

merged 2 commits into from
Jul 4, 2024

Conversation

AmanRaj1608
Copy link
Collaborator

@AmanRaj1608 AmanRaj1608 commented Jul 4, 2024

Issue ticket number and link

Checklist before requesting a review

  • I have performed a self-review of my code
  • If it is a core feature, I have added thorough tests.
  • Do we need to implement analytics?
  • Will this be part of a product update? If yes, please write one phrase about this update.

fix: update api interface

fix: remove graceful shhutdown
@AmanRaj1608 AmanRaj1608 merged commit be1a4eb into feat/settlement-engine Jul 4, 2024
1 check passed
@AmanRaj1608 AmanRaj1608 deleted the feat/api-routes branch July 4, 2024 10:31
AmanRaj1608 added a commit that referenced this pull request Jul 11, 2024
* feat: socket::generate_route_transactions

* feat: update engine spec and api validation

* fix: merge the route struct

* merge upstream changes

* refactor: engine.rs

* refactor: BridgeResult

* feat: add config api

fix: update api interface

fix: remove graceful shhutdown

* feat: api routes (#16)

fix: update api interface

fix: remove graceful shhutdown

* feat: add user address to extract_balance_data function

* feat: Settlement Engine

* feat: Read Approval Data from Chain

* feat: RPC URL for chains in config

* test: Approval Logic

* feat: Settlement Engine

* fix: exclude sol! block from doctest

* temp: Settlement Engine Service Controller Integration

* temp-fix: 'parameter may not live long enough'

* temp-fix: 'implementation of fnOnce not general enough'

* fix: Future is not send

* fix: Disable doc tests

---------

Co-authored-by: amanraj1608 <[email protected]>
Co-authored-by: Aman Raj <[email protected]>
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.

2 participants