Consider updating the Claimed event. Would be great if we could:
add merkleRoot- it would help to identify if a reward was claimed for a current merkle root. In that case, the additional call to a contract to check if a reward is available to claim will be unnecessary because we receive the merkleRoot in event data. Maybe it makes sense to make this parameter as indexed,
rename the account to stakingProvider just to clarify because we will generate the merkle tree per staking provider,
add the indexed beneficiary param because the tokens go to the beneficiary. Not necessary, but maybe it makes sense.
Consider updating the
Claimedevent. Would be great if we could:merkleRoot- it would help to identify if a reward was claimed for a current merkle root. In that case, the additional call to a contract to check if a reward is available to claim will be unnecessary because we receive themerkleRootin event data. Maybe it makes sense to make this parameter as indexed,accounttostakingProviderjust to clarify because we will generate the merkle tree per staking provider,beneficiaryparam because the tokens go to the beneficiary. Not necessary, but maybe it makes sense.