zeitgeist icon indicating copy to clipboard operation
zeitgeist copied to clipboard

Use only one type of auto-resolve

Open maltekliemann opened this issue 2 years ago • 3 comments

We currently have two types of auto-resolve, one for reported markets and one for disputed markets. Unless there's a good reason for this, we should get rid of this and just put all market ids in one map.

maltekliemann avatar Dec 06 '22 20:12 maltekliemann

@Chralt98 Any reason why this is the case?

maltekliemann avatar Oct 25 '23 00:10 maltekliemann

I think we already had an issue for this to merge MarketIdsPerDisputeBlock and MarketIdsPerReportBlock together.

Chralt98 avatar Nov 10 '23 14:11 Chralt98

I do think this issue is high effort and low return. I think it requires a storage migration. However, the benefit would be less code. But in general, this item on the list seems to be a "we will always have something better to do".

Chralt98 avatar Nov 10 '23 14:11 Chralt98