-
Notifications
You must be signed in to change notification settings - Fork 4
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
fix: don't throw ipfs errors, use defaults #149
Conversation
✅ Deploy Preview for groups-ui-storybook ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for regen-groups-ui ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can also test using the deploy preview connected to mainnet here.
Any ideas why the proposal is not showing up when we hit this error? I think we want to have a fallback in place and show the proposal. It looks like this solves the immediate issue though with the page not loading.
I'm ok with addressing the fallback separately if not a quick fix.
@ryanchristo it was showing up for me when i tested locally, i’ll take a better look but it may just be a separate issue. i’m kind of confused by what’s showing in your logs, unless i’m mistaken i thought the new log statement i added would have shown up there. oh you know what, maybe it was because i tested the metadata as precisely ipfs://CID whereas the metadata in the proposal Sam added might have had quote characters around it. i’ll look at more and see if that was it, and if so i’ll just do a fix here |
You should also be able to test locally with the proposal on mainnet: http://localhost:5173/6 |
Actually you would hit a CORS error if you did this with the production indexer so nvm. |
nevermind. that's not the case. duh |
we can disable CORS temporarily on the server, and that should make it work, and allow better debugging |
it looks like a possible explanation for why the proposals are not showing up here: Request/response for first policy: Request/response for second policy: it's fairly obvious that's a bug because we get some historical proposals, so those should be showing. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tACK. Sounds like a separate issue. This pr looks good to me.
Opened an issue for the missing proposals: |
Closes: #146
Testing:
Go to the page for the group in the groups-ui, observe that the group page and proposal page loads successfully.
Checkout
dev
, repeat 1-3, observe the failure.