You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The way current reputation oracle works is that when you make a request to getReputation, behind the scenes, it not only fetches the current user's reputation value, but also the proofs (key, branchMask, siblings, ...) for which it needs to load and process the whole reputation state, which is a heavy operation.
Currently we do the above for all places we need the reputation, both for display (where we don't actually need proofs) and both for sending a transaction (where we do).
To optimize this, and ensure the reputation, at least for display purposes loads faster (a lot of dapp hang time currently is because of the reputation orancle endpoint being slow), Alex introduced a new enpoint for the reputation miner, which just returns the value we need to display, without the proofs.
I'll integrate that into colonyJS, most likely as getReputationLight or getReputationValue, and then this issue needs to refactor the current reputation display in the dapp.
Components will use the new "light" reputation display (well, i guess this goes in the resolver actually...).
Sagas will still use the old, "heavy" way of fetching the reputation since in there we also require the proofs.
I'll leave a comment in here after I've built the new client, and that this can be tackled.
The text was updated successfully, but these errors were encountered:
The way current reputation oracle works is that when you make a request to
getReputation
, behind the scenes, it not only fetches the current user's reputation value, but also the proofs (key
,branchMask
,siblings
, ...) for which it needs to load and process the whole reputation state, which is a heavy operation.Currently we do the above for all places we need the reputation, both for display (where we don't actually need proofs) and both for sending a transaction (where we do).
To optimize this, and ensure the reputation, at least for display purposes loads faster (a lot of dapp hang time currently is because of the reputation orancle endpoint being slow), Alex introduced a new enpoint for the reputation miner, which just returns the value we need to display, without the proofs.
I'll integrate that into
colonyJS
, most likely asgetReputationLight
orgetReputationValue
, and then this issue needs to refactor the current reputation display in the dapp.Components will use the new "light" reputation display (well, i guess this goes in the resolver actually...).
Sagas will still use the old, "heavy" way of fetching the reputation since in there we also require the proofs.
I'll leave a comment in here after I've built the new client, and that this can be tackled.
The text was updated successfully, but these errors were encountered: