[SUBGRAPH BUG] flowDistributionUpdatedEvents not being picked up by the graph about protocol-monorepo HOT 7 CLOSED

Agree, we need to handle this mappings in metadata.
The numerical chainId is the only deterministic and unique identifier we can rely on across the ecosystem.
Going forward we should try to minimize the amount of non-SF-specific information we need to manually maintain in metadata in order to keep maintenance overhead of shuffling networks low.
Can be done by adding a postprocessing step to metadata (or just add to the existing build script) which pulls in relevant metadata provided by others, e.g. https://github.com/wagmi-dev/viem/tree/main/src/chains/definitions , and merges that with SF-specific metadata.

from protocol-monorepo.

attempting to redeploy subgraph: https://github.com/superfluid-finance/protocol-monorepo/actions/runs/6865195493

it is quite a confusing issue otherwise

from protocol-monorepo.

looks like the optimism-goerli subgraph is quite outdated, I would reckon that this is related to the issue.

from protocol-monorepo.

do we redeploy?

from protocol-monorepo.

I tried to but there was an issue in buildNetworkConfig.ts due to inconsistencies between what we've decided to go with in terms of canonical names in shortName or name in metadata.

Addressing this root issue, once and for all here: #1748.

Placing the canonical name in metadata once and then we can just forget about it and use the canonical name instead.

from protocol-monorepo.

fixed and re-synced

from protocol-monorepo.

#1747 (comment)

from protocol-monorepo.