Skip to content
This repository has been archived by the owner on Nov 20, 2024. It is now read-only.

Duplicated transaction displayed in Batch details screen #260

Open
tclemos opened this issue Jul 19, 2021 · 1 comment
Open

Duplicated transaction displayed in Batch details screen #260

tclemos opened this issue Jul 19, 2021 · 1 comment
Labels
type: bug Something isn't working

Comments

@tclemos
Copy link

tclemos commented Jul 19, 2021

Summary of Bug

Hey people, looks like I found something weird in the testnet explorer.
I was double-checking the airdrop payments and figured out that some transactions were listed duplicated in the explorer website.

the block is this one: https://explorer.testnet.hermez.io/batch/6236
the duplicated transactions are: 0x02213bd36479cf0028428ca9f1441d3b37a026039c4c466af64e490f51763a0a2d and 0x02aa79146d7b1bd06803c6d911a538c7a83eacc219a80292d27dbcc9f39f44e677

I've already double-checked the API called by the front end, this is the URL being called to retrieve the transaction history: https://api.testnet.hermez.io/v1/transactions-history?batchNum=6236&order=DESC&limit=20

Accordingly to the API response, the transactions are not duplicated in the response, so I'm suspecting something is happening in the front-end logic.

Steps to Reproduce

Access this url: https://explorer.testnet.hermez.io/batch/6236
Check the transaction list, there will be two lines for the transactions 0x02213bd36479cf0028428ca9f1441d3b37a026039c4c466af64e490f51763a0a2d and 0x02aa79146d7b1bd06803c6d911a538c7a83eacc219a80292d27dbcc9f39f44e677

Browser information

  • Browser: Chrome
  • Device/OS: Desktop/Windows 10

Additional Information & Screenshots:

image

@tclemos tclemos added the type: bug Something isn't working label Jul 19, 2021
@invocamanman
Copy link

invocamanman commented Jul 19, 2021

As an added information when there are more than 20 items in the same batch (20+ transactions batch), it fetch the following transactions using the following query https://api.testnet.hermez.io/v1/transactions-history?batchNum=6236&fromItem=34293&order=DESC&limit=20.

The fromItem is calculated incorrectly, currently the last itemId in the query https://api.testnet.hermez.io/v1/transactions-history?batchNum=6236&order=DESC&limit=20 is 34292, but the batch explorer call withfromItem=34293 when it should be 34291.
That's why in all the batches that has 20+ transactions, the (19 - 21) and (20 - 22) transactions are repeated

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type: bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants