-
Notifications
You must be signed in to change notification settings - Fork 24
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: quotes-server #430
Closed
Closed
feat: quotes-server #430
Changes from 27 commits
Commits
Show all changes
37 commits
Select commit
Hold shift + click to select a range
0ff4862
feat: quotes-server
bodymindarts b6dc7ca
chore: copy currency code to quotes
bodymindarts fdf2eb1
chore: more scoping under currency_exchange
bodymindarts cb680e8
refactor: rename currency_exchange -> price
bodymindarts d1ff1a7
chore: quotes app init
bodymindarts 5f45af3
test: improve mixer test with Dummys
bodymindarts 5cbbd0c
chore: price use case
bodymindarts c7ba7a6
test: all use cases of PriceCaclculator
bodymindarts 8121681
refactor: rename proto api
bodymindarts 990ea90
feat: use order book in quotes server
thevaibhav-dixit 29e0bdb
chore: add entity folder for quotes-server
thevaibhav-dixit b95e5c3
feat: add entity events and repo
thevaibhav-dixit 539a22b
chore: quotes_server in cli
thevaibhav-dixit 3331a64
chore: construct quote
thevaibhav-dixit 2f6d6db
chore: run sqlx prepare
thevaibhav-dixit e3b77c7
feat: add sats/cents for sell/buy
thevaibhav-dixit 00ba01b
refactor: index only required fields
thevaibhav-dixit 04cd534
chore: address pr reviews
thevaibhav-dixit 76b52f7
chore: complete server_side code
thevaibhav-dixit 13c272f
chore: address pr review
thevaibhav-dixit 5388ba9
chore: fix quote table
thevaibhav-dixit 1300d11
feat: add quote_buy_template
thevaibhav-dixit ebc5f6f
feat: add templates for quotes
thevaibhav-dixit f145658
refactor: remove revert templates
thevaibhav-dixit df81c15
fix: use proper error message
thevaibhav-dixit 5b6360c
feat: implement accept_quote use case
thevaibhav-dixit 81550fb
fix: rename stablesats_quote -> stablesats_quotes
thevaibhav-dixit 8fab192
fix: add update fn for quotes
thevaibhav-dixit edecb90
test: add quotes_app test
thevaibhav-dixit b16e0e7
refactor: add config based expiration_interval
thevaibhav-dixit f45cf99
refactor: remove redundant errors
thevaibhav-dixit 2b85777
refactor: add accepted and expiration related errors
thevaibhav-dixit da7c5db
test: cover various cases for quotes_app test
thevaibhav-dixit 7688148
refactor: checks on entity and make txn's atomic
thevaibhav-dixit 3dc60d9
chore: add mark_events to entity_events
bodymindarts 5276163
fix: use consistent style
thevaibhav-dixit bc940f4
feat: include cent/sat spread in quote(first-iteration)
thevaibhav-dixit File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
does it make sense to be different than the price server?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Easier to immutably replace rather than try to merge legacy with new server IMO