-
Notifications
You must be signed in to change notification settings - Fork 3
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
Comments
@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 |
Can confirm automated E2E testing for Econ gov UI failed attempting to vote |
Honing in on where. we make the vstorage calls:
managerId
|
Manager ids are getting fetched from vault here:
|
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? |
Changing url to {
"children": [
"manager0",
"manager1",
"manager2",
"manager3",
"manager4"
],
"pagination": null
} Question is why are we not able to properly parse these managers? |
I hopped into a debugger. Looks like we are always getting a
|
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 |
Moving on to why the votes aren't visible. Coordinating with @dckc, this may be related to a contract failure. Investigating... |
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) |
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. |
This seems to be fixed now based on manual testing in emerynet and our automated tests |
I think the silent failure is worth fixing... or at least tracking. @otoole-brendan I suggest refining the title to something like:
|
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:
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
The text was updated successfully, but these errors were encountered: