Mul 1865 new logic and interface for eos tx #41
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
change file name from `EOS` to `eos` move struct EosTransactionAction to new file move struct EosTransactionTransferAction to new file
PashaKlybik
force-pushed
the
MUL-1865-New-logic-and-interface-for-EOS-Tx
branch
5 times, most recently
from
408c917
to
c64af94
Compare
Create EOS transfer transaction builder
PashaKlybik
force-pushed
the
MUL-1865-New-logic-and-interface-for-EOS-Tx
branch
from
c64af94
to
728d8cd
Compare
Enmk
suggested changes
Apr 29, 2019
| @@ -46,6 +46,16 @@ const EosChainId EOS_MAINNET_CHAIN_ID = {0xac, 0xa3, 0x76, 0xf2, 0x06, 0xb8, 0xf | |||
| const std::array<uint8_t, 32> EOS_ZERO_SHA256 = {0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, | |||
| 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00}; | |||
|
|
|||
| void putVariableUInt(EosBinaryStream* stream, uint64_t value) | |||
There was a problem hiding this comment.
Please update signature:
WriteToStream(uint64_t value, EosBinaryStream* stream)
| { | ||
| do | ||
| { | ||
| uint8_t input = static_cast<uint8_t>(value & 0x7f); |
| @@ -46,6 +46,16 @@ const EosChainId EOS_MAINNET_CHAIN_ID = {0xac, 0xa3, 0x76, 0xf2, 0x06, 0xb8, 0xf | |||
| const std::array<uint8_t, 32> EOS_ZERO_SHA256 = {0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, | |||
| 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00}; | |||
|
|
|||
| void putVariableUInt(EosBinaryStream* stream, uint64_t value) | |||
| { | |||
There was a problem hiding this comment.
Please put a link to reference documentation here
| @@ -86,6 +87,10 @@ TransactionBuilderPtr EosFacade::make_transaction_builder( | |||
| EOS_TRANSACTION_BUILDER_UPDATEAUTH, | |||
There was a problem hiding this comment.
IIRC, we now have an additional overload that takes string as builder name, please add it here.
| @@ -97,7 +98,8 @@ uint64_t EosName::get_data() const | |||
|
|
|||
| std::string EosName::get_string() const | |||
| { | |||
| THROW_EXCEPTION2(ERROR_FEATURE_NOT_IMPLEMENTED_YET, __FUNCTION__); | |||
| return m_data_string; | |||
| //THROW_EXCEPTION2(ERROR_FEATURE_NOT_IMPLEMENTED_YET, __FUNCTION__); | |||
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in 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.