seeds_light_wallet icon indicating copy to clipboard operation
seeds_light_wallet copied to clipboard

A vouched for user still shows up as available for vouching

Open buildc0de opened this issue 3 years ago • 1 comments

Reported by Sorin in Discord.

To Reproduce Steps to reproduce the behavior:

  1. Login as gradinagruiu.
  2. Attempt to vouch for fastforward1.

Current behavior Observe the Error when vouching for a member error.

Expected behavior Since the user has already been vouched for, it is expected to be shown in Vouched for list

Screenshots See the Discord thread.

Platform & App Version

  • Android
  • 2.9.0 (80)

Additional context As reported by @n13:

❯ cleosm get table accts.seeds accts.seeds vouches --index 2 --key-type name --lower fastforward1
{
  "rows": [{
      "id": 10344,
      "account": "fastforward1",
      "sponsor": "gradinagruiu",
      "vouch_points": 16

buildc0de avatar Jun 21 '22 00:06 buildc0de

I think that maybe the vouch list became too long, and the wallet only fetches a subset?

This may be a problem for many areas of the wallet that load a list from the contract tables

The number of items that the contract tables return is not guaranteed to be complete - in fact it can in theory be only 1 item with "more"="true" as an indication that there's more to load.

I think we need a general "load it all" table load function

Anyway - if that is indeed the problem.

n13 avatar Jun 29 '22 16:06 n13