Добавить генерацию для operationID при загрузки средств в канал транзакций из внешних кошельков
На данном этапе в канале транзакций при добавление и вводе средств с внешних кошельков отсутствует часть записи отвечающая за operationID
. Т.е. по сути запись в реестр выглядит следующим образом
// OUT
if err := (&TransactionRecord{
Key: outKey,
CurrencyType: cryptocurrencyStore.CurrencyType,
User: UserAccountInfo{
// Keep default values for time being, has to be replaced with
// call to system chaincode to obtain system user and account information
UserID: "systemUserID",
AccountInfo: AccountInfo{
cryptocurrencyStore.OutAccountInfo.WalletID,
cryptocurrencyStore.OutAccountInfo.AccountID,
},
},
Amount: cryptocurrencyStore.Amount,
OperationID: "", // TODO: Where to get the operation ID?
OperationType: TransferAmountToUser,
ProcessType: DepositFunds,
HashCode: "", // TODO: Ask what is the meaning of the hash code here
DependencyKey: inKey,
Timestamp: tt,
}).Save(adapter); err != nil {
return err
}
и зеркальная часть транзакции
// IN
if err := (&TransactionRecord{
Key: inKey,
CurrencyType: cryptocurrencyStore.CurrencyType,
// Key and the account information should be aligned together
// TODO: Check whenever inKey and user information filled as appropriate
User: UserAccountInfo{
UserID: cryptocurrencyStore.InAccountInfo.UserID,
AccountInfo: AccountInfo{
WalletID: cryptocurrencyStore.InAccountInfo.WalletID,
AccountID: cryptocurrencyStore.InAccountInfo.AccountID,
},
},
Amount: cryptocurrencyStore.Amount,
OperationID: "",
OperationType: TransferAmountToUser,
ProcessType: DepositFunds,
HashCode: "", // TODO: Ask what is the meaning of the hash code here
DependencyKey: outKey,
Timestamp: tt,
}).Save(adapter); err != nil {
return err
}
т.е. operationID
отсутствует в обоих случаях.
Необходимо разобраться и решить каким образом этот ID будет генерироваться и поддерживаться в системе. Как одна из возможных альтернатив, может быть добавление транзакций ввода средств на уровне scheudler'а который и будет учитывать и обеспечивать уникальность.
@yandrevv почитай пожалуйста и проверь.