Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Active Votes nor historical votes visible on 'Vote' tab #112

Open
otoole-brendan opened this issue Aug 23, 2024 · 14 comments
Open

Active Votes nor historical votes visible on 'Vote' tab #112

otoole-brendan opened this issue Aug 23, 2024 · 14 comments
Assignees
Labels
bug Something isn't working

Comments

@otoole-brendan
Copy link

Describe the bug
While smoke-testing Emerynet as part of the Auction and Vaults core eval - I attempted to raise the mint limit for ATOM vaults. I was able to propose the parameter change but when I went to vote on it it never appears.

Screenshot of console doesn't reveal anything obvious although another set of eyes here would be beneficial

To Reproduce
Steps to reproduce the behavior:

  1. Go to '...'
  2. Click on 'Vaults' tab.
  3. Propose any parameter change
  4. Click on 'Vote' tab

Expected behavior
The active proposed parameter change should appear and I should be able to vote on it. I should be able to see past parameter proposals also.

Screenshots
image

@otoole-brendan otoole-brendan added the bug Something isn't working label Aug 23, 2024
@otoole-brendan otoole-brendan self-assigned this Aug 23, 2024
@otoole-brendan
Copy link
Author

otoole-brendan commented Aug 23, 2024

@ivanlei was able to propose a PSM param change and see that vote (And vote on it) so doesn't look like it's affecting PSM param votes, only Vaults https://emerynet.explorer.agoric.net/agoric/tx/FCE824D8D15BD4892542EC1F66E27AA49F4579B13B034DBB9FE84FD7150F9C41
image

@otoole-brendan
Copy link
Author

Current assumption is dapp is somehow building an incorrect path for reading VaultManagers in vstorage, we're seeing lots of errors like:
image

@otoole-brendan
Copy link
Author

Can confirm automated E2E testing for Econ gov UI failed attempting to vote

@iomekam
Copy link

iomekam commented Aug 23, 2024

Honing in on where. we make the vstorage calls:

export default function VaultParamChange(props: Props) {
. It looks like we aren't correctly finding the correct managerId

@iomekam
Copy link

iomekam commented Aug 23, 2024

Manager ids are getting fetched from vault here:

const { data: vaultKeys, status: vaultKeysStatus } = usePublishedKeys(
. I'm going to try some API calls on Emerynet to see what's current in vstorage

@iomekam
Copy link

iomekam commented Aug 23, 2024

CHAIN_URL=https://emerynet.api.agoric.net
URL="$CHAIN_URL/agoric/vstorage/data/published.vaultFactory.managers"  
curl -s $URL

Output:

{
  "value": ""
}

Seems suspicious 🤔.

Let's try this on mainnet....

CHAIN_URL=https://emerynet.api.agoric.net
URL="$CHAIN_URL/agoric/vstorage/data/published.vaultFactory.managers"  
curl -s $URL

Output:

{
  "value": ""
}

They are the same. Is this the right url to query?

@iomekam
Copy link

iomekam commented Aug 23, 2024

Changing url to URL="$CHAIN_URL/agoric/vstorage/children/published.vaultFactory.managers seems to have worked. Both chains are returning

{
  "children": [
    "manager0",
    "manager1",
    "manager2",
    "manager3",
    "manager4"
  ],
  "pagination": null
}

Question is why are we not able to properly parse these managers?

@iomekam
Copy link

iomekam commented Aug 23, 2024

I hopped into a debugger. Looks like we are always getting a null for mangerId.

const [managerId, setManagerId] = useState(null);
. Diving deeper

@iomekam
Copy link

iomekam commented Aug 23, 2024

Looks like this may be expected behavior. This will error out when we first open the page, but once we select a manger, it's happy

@iomekam
Copy link

iomekam commented Aug 23, 2024

Moving on to why the votes aren't visible. Coordinating with @dckc, this may be related to a contract failure. Investigating...

@iomekam
Copy link

iomekam commented Aug 23, 2024

From the perspective of dapp-econ-gov, I don't think anything is broken. We have traction that the core issue is related to Agoric/agoric-sdk#5200.

p.s. specifically Agoric/agoric-sdk#5200 (comment)

@dckc
Copy link
Member

dckc commented Aug 23, 2024

Well, one broken thing about dapp-econ-gov is that proposing to change the mint limit fails silently.

The smartWallet reports the result of the offer to vstorage, including a vat terminated error, but this dapp doesn't report it.

@samsiegart
Copy link
Contributor

This seems to be fixed now based on manual testing in emerynet and our automated tests

@dckc
Copy link
Member

dckc commented Aug 29, 2024

I think the silent failure is worth fixing... or at least tracking.

@otoole-brendan I suggest refining the title to something like:

  • errors from proposing parameter changes are not reported

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants