-
Notifications
You must be signed in to change notification settings - Fork 102
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
tarodb: add chain_fees
to chain_txns
#184
Conversation
@@ -0,0 +1,2 @@ | |||
ALTER TABLE chain_txns ADD chain_fees INTEGER; | |||
|
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.
We're still in alpha and already know that users will need to drop their DB state with the next release. So this can be added directly to the original migration file.
@guggero: review reminder |
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.
Nice and clean, LGTM 🎉
Populate the new field using the funded PSBT data.
Store on-chain fees as a separate column so we can directly execute queries in our database using this value. Not all chain tx will have this field populated. We will store the fees only for the tx that we create (minting and trasfer operations but not importing).
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.
LGTM 🕷
Store how much we spent in on-chain fee for minting/transfers operations.
Fixes #144
Affects #177