Inconsistency in the way FACTORY_ADDRESS is defined and used

[llopez]

There seems to be an inconsistency in how FACTORY_ADDRESS is defined and consumed across the codebase. In some places, it is expected to be of type Address, while in others it is treated as a plain string

if we do this
https://github.com/Uniswap/v3-subgraph/blob/main/config/matic/chain.ts#L3

it fails in this line
https://github.com/Uniswap/v3-subgraph/blob/main/src/common/constants.ts#L14