zeitgeist
zeitgeist copied to clipboard
Release event with only changed fields when market is edited
It would be good to have event PredictionMarkets.MarketEdited
which only has fields whose data got updated by the user. This should allow lay person to identify the changed data and reduce work for downstreams especially.
Presently subsquid would check every field on the market schema before and after edit to track changes.
Example: As seen for marketId:535
on testnet, its difficult to make out what changed post edit.
The solution should be on the Squid side imo, as the actual metadata lives in IPFS it should fetch the data and diff it. We could also enforce providing the actual metadata as a bytearray to the on-chain call and checking if it matches the metadata, but that introduces unnecessary load on the chain and raises the transaction cost, as the transaction gets bigger and computation time increases.
Closing because we're removing market editing completely.