-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
R4R: New broadcast command #2240
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #2240 +/- ##
========================================
Coverage 63.74% 63.74%
========================================
Files 140 140
Lines 8645 8645
========================================
Hits 5511 5511
Misses 2755 2755
Partials 379 379 |
2f85aaf
to
1cf2d5f
Compare
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.
Left some comments. Mostly naming, tests and docs. Logic seems fine.
Lets vote:
|
f4c46a7
to
f0c27ed
Compare
28afbda
to
8ee7a52
Compare
@alessio FYI, you don't need to squash while writing the PR, that can be done by the reviewer at the end. |
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.
Quick comments.
Implement broadcast command/REST endpoint to submit transactions generated offline with --generated-only and the sign command. Closes: #1954
8708c44
to
4c8cd38
Compare
Implement
broadcast
command/REST endpoint to submit transactions generated offline with--generated-only
and thesign
command.Closes: #1954
docs/
)PENDING.md
with issue #Files changed
in the github PR explorerFor Admin Use: