Factory address constant should be updated to a chain: address map

[JFrankfurt]

FACTORY_ADDRESS should be an AddressMap like WETH9 instead of a string constant that's invalid on some networks. 0x1F98431c8aD98523631AE4a59f267346ea31F984 is not deployed on Base and there have been some transactions to this address assuming it existed: https://basescan.org/address/0x1F98431c8aD98523631AE4a59f267346ea31F984

[JFrankfurt]

@zhongeric do me a solid bruh

[hensha256]

@JFrankfurt i think you should be using V3_CORE_FACTORY_ADDRESSES instead! FACTORY_ADDRESS should probably be deprecated to signal that.
@jsy1218 is that right?

[jsy1218]

That's right. We should mark v3 FACTORY_ADDRESS as deprecated.

[jsy1218]

@JFrankfurt @hensha256 addressed in #58