Pending update: mm2 -> kdf in coins file

[smk762]

ref: KomodoPlatform/coins#1121

[onur-ozkan]

Do we have anything missing in this repo for mm2->kdf replacement?

[cipig]

Do we have anything missing in this repo for mm2->kdf replacement?

does mm2 still check if the coins/tokens have "mm2": 1 set it coins file?

[onur-ozkan]

We haven't change mm2 naming in the RPC payloads yet, in fact we should make a smooth transition for that.

Created an issue to not forget about it: #2293

[cipig]

We haven't change mm2 naming in the RPC payloads yet, in fact we should make a smooth transition for that.

No need to rename, just remove the param mm2 completely. It wouldn't make a difference anyway, the result would be the exact same, since there are no coins without mm2 param set that also have electrums and everything else needed.

[onur-ozkan]

We rely on that field for certain logics.

[cipig]

We rely on that field for certain logics.

what kind of logic? all coins/tokens in current coins file have mm2:1 set, there are no coins without it.

[laruh]

We rely on that field for certain logics.

As I can see, right mow only EthActivationV2Request structure has mm2 field and we dont use it

Also, the adex_cli request structs have it, but we are going to remove this crate, aren't we? and it seems like mm2 field isn't used in adex_cli either.