evm-labels icon indicating copy to clipboard operation
evm-labels copied to clipboard
verified publisher icon dawsbot

incomplete pulling from optimism and potentially etherscan.

Open kylewandishin opened this issue 1 year ago • 1 comments

image this matches with our combined.json in /data/basescan which has a length of 6318



image this does NOT match our combined.json in /data/optimism which only has a length of 2467 which is 101 records short.

kylewandishin avatar Mar 26 '24 21:03 kylewandishin

Hey @kylewandishin I am only now seeing this and I simplified that file to only write combined.json files for labels which had more than one json in it.

In that process I removed this code you're mentioning here. Can you describe the issue or open a new PR to v1 that fixes this issue? https://github.com/dawsbot/evm-labels/commit/c3b8ada4ac02dcd9d5acddefcf5f345786d3162b#diff-25d84be0fe67b7bbd33fecabee07a00b177e2fea3c873955d89685bfd4f9af14R28

dawsbot avatar Mar 27 '24 17:03 dawsbot

I still don't understand this issue @kylewandishin - can you try again to explain what's happening simpler?

dawsbot avatar May 18 '24 19:05 dawsbot

@dawsbot this issue was incorrect. I realised just now that this failed to account for the max cap on tokens at 100 and the max cap on accounts

kylewandishin avatar May 19 '24 19:05 kylewandishin

i think we should close this.

kylewandishin avatar May 19 '24 19:05 kylewandishin