forked from empeje/core-eng
-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Showcase commits on contributions #13
Open
Web3RustDev
wants to merge
36
commits into
main
Choose a base branch
from
fund-script-both-ways
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
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
update: switched to old signer config because they weren't working, added regtest as network, modified coordinator config to work\nfix: some libraries not properly imported\ntemporary fix (needs to be changed back): ordered utxos by amount instead of confirmations since blocks aren't being mined and utxos don't have confirmations
update: stacks private key for coordinator, fixed function names for positive/negative removal vote
…ol's total spend per block update: changed to v2 contract with improvements and fixes
…txo response update: updated to latest version of stacks-sbtc fork from Trust Machines
update: tested voting out flow, commented functions that are not on smart contract so coordinator code works
update: checking if the signer sent the correct amount through his utxo, if not pox transaction is not created. Read-only call for amount on signer's tx. fix: coordinator failing due to an attempt to display a vector.
…height x fix: divided amount to script to number of miners
…t to pox update: fixed a few todos, removed unused ones
…20 minutes after the propose for removal call
…ept miners in pool update: added function to list the imported descriptors so they don't get re-imported. fix: user to script tx creation, signing now accepts multiple inputs
…ator update: uncommented used line, removed unused code in refund tx creation and signing fix: listdescriptors is now called on the current wallet
…least 1 transaction
…s and amount update: added is-proposed-for-removal sc read-only call and checking it every 5 minutes while waiting to vote miners out to be sure that bad actors are removed, bitcoin to regtest network on testnet config (for testing), removed unused code part
…local nonce and increment it with 1 in order to fix the 'conflicting nonce' error
…o amount is greater (instead of smaller) than the amount needed
…fore voting him out fix: moved first nonce api call out of loop (which was causing it to still give ConflictingNonce error), fixed is_proposed_for_removal call returning an error update: changed unwraps and expects to unwrap_or or match for error handling so the code won't panic if an error comes, removed print for whom coordinator votes out from degen_base_coordinator
…e correct ones fix: pox transaction sending more money than needed back to scripts.
…he automated operations thread
…put address and amount from a txid
…onfig fix: variables for transaction checking being initialized with 1 element, causing the check to fail
…r with a given amount add: check the blocks for which the coordinator didn't create a pox transaction (log to file) update: removed unused functions/imports
So they don't get voted out of pool
…rom local to github
# for free
to join this conversation on GitHub.
Already have an account?
# to comment
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.
No description provided.