Skip to content

MyMeta: xDai and Polygon DAOs frontend#656

Merged
alalonde merged 3 commits into
developfrom
feat/xdai-polygon-dao-fe
Jun 24, 2021
Merged

MyMeta: xDai and Polygon DAOs frontend#656
alalonde merged 3 commits into
developfrom
feat/xdai-polygon-dao-fe

Conversation

@mattcan
Copy link
Copy Markdown

@mattcan mattcan commented Jun 14, 2021

Related #643

Closes #549


This change updates the UI to better support the xDai and Polygon DAO data feed as well as bring a style that better matches the NFT gallery.

Screenshots

New player memberships screen:

656-playerscreen

'View all' modal:

656-daomodal

mattcan added 3 commits June 14, 2021 11:57
I don't think such a replacement should occur in the backend unless we
are creating our own models (not the case here). Therefore, this will be
handled on the frontend.
This matches the NFT gallery functionality
@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 14, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/metafam/the-game/6ZXqMsJxyKQoJZ8f84mGMJXzARWt
✅ Preview: Failed

[Deployment for a54846d failed]

@mattcan
Copy link
Copy Markdown
Author

mattcan commented Jun 14, 2021

Could someone please share the Vercel error? It seems to work locally with NEXT_PUBLIC_GRAPHQL_URL=https://api.metagame.wtf/v1/graphql.

@dan13ram
Copy link
Copy Markdown
Member

dan13ram commented Jun 16, 2021

@mattcan Here are the logs:


11:34:13.163 | @metafam/web: info  - Collecting page data...
-- | --
11:34:15.710 | @metafam/web: info  - Generating static pages (0/189)
11:34:18.788 | @metafam/web: Error occurred prerendering page "/players". Read more: https://nextjs.org/docs/messages/prerender-error
11:34:18.788 | @metafam/web: Error: Error serializing `.urqlState["3400386408"].error.graphQLErrors[0].path` returned from `getStaticProps` in "/players".
11:34:18.790 | @metafam/web: Reason: `undefined` cannot be serialized as JSON. Please use `null` or omit this value.
11:34:18.790 | @metafam/web:     at isSerializable (/vercel/path0/node_modules/next/dist/lib/is-serializable-props.js:7:95)
11:34:18.790 | @metafam/web:     at /vercel/path0/node_modules/next/dist/lib/is-serializable-props.js:7:490
11:34:18.790 | @metafam/web:     at Array.every (<anonymous>)
11:34:18.790 | @metafam/web:     at isSerializable (/vercel/path0/node_modules/next/dist/lib/is-serializable-props.js:7:291)
11:34:18.790 | @metafam/web:     at /vercel/path0/node_modules/next/dist/lib/is-serializable-props.js:7:769
11:34:18.790 | @metafam/web:     at Array.every (<anonymous>)
11:34:18.790 | @metafam/web:     at isSerializable (/vercel/path0/node_modules/next/dist/lib/is-serializable-props.js:7:706)
11:34:18.790 | @metafam/web:     at /vercel/path0/node_modules/next/dist/lib/is-serializable-props.js:7:490
11:34:18.790 | @metafam/web:     at Array.every (<anonymous>)
11:34:18.791 | @metafam/web:     at isSerializable (/vercel/path0/node_modules/next/dist/lib/is-serializable-props.js:7:291)


I'm afraid though that afaik the issue is not related to your code. I have the builds failing locally as well.
It fails for both https://api.metagame.wtf/v1/graphql and https://api-staging.metagame.wtf/v1/graphql

Could you double check if your builds are passing locally?

@alalonde
Copy link
Copy Markdown
Contributor

Local build worked fine for me, I believe the failed build is due to some data discrepancy in the staging environment.

@alalonde alalonde merged commit 08198ec into develop Jun 24, 2021
@alalonde alalonde deleted the feat/xdai-polygon-dao-fe branch June 24, 2021 17:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[MyMeta] Add support for xDAI & Polygon DAOs

3 participants