Skip to content

Conversation

iltoga
Copy link
Contributor

@iltoga iltoga commented May 14, 2020

Description

pack of bug fixes.

nawikart and others added 5 commits May 14, 2020 08:25
…816)

* first commit

* fix: Missing Rollback When BackupMempool Fail in PoppedOffBlock 

* change BackupMempols --> BackupMempools
* GetAccountDatasets

* GetAccountDataset
* ignore .bak like genesis.go.bak :)

* expiring pending transactions by height and undo apply unconfirmed

* update readme about multis sig generator

* not allow filter by pending and expired

* finish query logic to, so now (block_height+min_rollback) = current_block_height

* defer func for rows close and matches the others about query logic

* add log instance into transaction core service, ignore log level

Co-authored-by: Gede Sukra Widhyawan <[email protected]>
Co-authored-by: Andy Shi <[email protected]>
* TestBlockHandler_GetBlock

* TestBlockHandler_GetBlocks

* fix captLocal

* fix gocritic
* TestMempoolTransactionHandler_GetMempoolTransaction

* TestMempoolTransactionHandler_GetMempoolTransactions
@iltoga iltoga requested review from astaphobia, andy-shi88, capt4ce and sukrawidhyawan and removed request for astaphobia May 14, 2020 02:08
@iltoga iltoga self-assigned this May 14, 2020
@iltoga iltoga added the BUG Something isn't working label May 14, 2020
@iltoga iltoga added this to the Milestone 17 milestone May 14, 2020
* #832 add one more persistance time cycle to check to avoid node stuck in small network

* #829 include escrow detail of transaction in GetTransactionByBlockID

* #829 include escrow detail of transaction in GetTransactionByBlockID

* #833 add copy of block to broadcast before deleting the transactions

* #829 fix test

* #829 add test for escrow transaction query

* #833 broadcasted wrong variable

* #829 refactor escrow transaction query to be more generic
@iltoga iltoga merged commit c669496 into experimental May 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BUG Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants