TheGame icon indicating copy to clipboard operation
TheGame copied to clipboard

[MetaOS] Rename xdai chain to gnosis chain

Open dan13ram opened this issue 2 years ago • 2 comments

What would you like to be added?

Replace ALL occurrences of names/logos/images/icons of "xdai chain" with "gnosis chain".

Why is this needed?

xDai chain was recently renamed to Gnosis Chain. Read more: https://www.xdaichain.com/about-gc/news-and-information/xdai-gnosis-merger-ama

dan13ram avatar Feb 28 '22 12:02 dan13ram

Ahoy,

  1. Would a search and replace through all the files be enough to find all the places that need to be changed?
  2. Are new logos/images/icons available, or would they need to be made?

I think I could do this work.

HHH-GH avatar Jun 04 '22 06:06 HHH-GH

I'll take this one.

I won't get on to it immediately, but if I don't see any objections/suggestions by the time I do get on to it I will proceed as follows:

  1. Go through all the files that mention 'xdai' as per https://github.com/MetaFam/TheGame/search?p=1&q=xdai
  2. Text updates/icon updates for things that appear on the frontend as text or icons.
  3. No updates for things involved in the backend

At a quick glance, on the frontend there are just a few text updates, plus a new SVG icon and component to be created.

On the backend, I'm seeing stuff like daoHausXdaiGraphqlURL in packages/backend/src/config.ts - I'm not planning to touch any of that.

HHH-GH avatar Aug 04 '22 08:08 HHH-GH

On the backend, I'm seeing stuff like daoHausXdaiGraphqlURL in packages/backend/src/config.ts — I'm not planning to touch any of that.

I would think it's safe to change this to daoHausGnosisGraphqlURL and the associated environment variable to $DAOHAUS_GNOSIS_GRAPHQL_URL since the name of the chain housing the DAOHaus DAOs is Gnosis.

dysbulic avatar Aug 23 '22 08:08 dysbulic

Another option is to write the DAOHaus Graphs as a single indexed object:

  daoHausGraphQLURL: {
    mainnet: parseEnv(
      process.env.DAOHAUS_GRAPHQL_URL,
      'https://api.thegraph.com/subgraphs/name/odyssy-automaton/daohaus',
    ),
    polygon: parseEnv(
      process.env.DAOHAUS_POLYGON_GRAPHQL_URL,
      'https://api.thegraph.com/subgraphs/name/odyssy-automaton/daohaus-matic',
    ),
    gnosis: parseEnv(
      process.env.DAOHAUS_GNOSIS_GRAPHQL_URL,
      'https://api.thegraph.com/subgraphs/name/odyssy-automaton/daohaus-xdai',
    ),
  },

Then, if the GraphQL structure is the same, they could be easily iterated over, and adding new ones is simpler as well.

dysbulic avatar Aug 23 '22 08:08 dysbulic

I'll take a look to see how that all comes together, thanks. (Will probably do the easy frontend bits first)

HHH-GH avatar Aug 24 '22 12:08 HHH-GH