[R4R]Field: price of TradingPair change to list_price

[unclezoro]

Description

list_price is more meaningful.

Rationale

no

Example

add an example CLI or API response...

Changes

Notable changes:
This may affect downstream service.
@ackratos @xzjcool @erhenglu @jiangkai-rq please note.

Preflight checks

• build passed (make build)
• tests passed (make test)
• integration tests passed (make integration_test)
• manual transaction test passed (cli invoke)

Already reviewed by

...

Related issues

... reference related issue #'s here ...

[darren-liu]

really need to confirm with the downstream about when to merge and release.

[rickyyangz]

incompatible change! Need to test and confirm whether we need fork logic.

[unclezoro]

incompatible change! Need to confirm when to merge, or use the fork logic...

I will give it a test

[unclezoro]

incompatible change! Need to test and confirm whether we need fork logic.

I did a test, amino marshall do not concern with the field name. After upgrade, the already exist markets are still readable. One thing should notice is that we need update bnbcli too.

[rickyyangz]

incompatible change! Need to test and confirm whether we need fork logic.

I did a test, amino marshall do not concern with the field name. After upgrade, the already exist markets are still readable. One thing should notice is that we need update bnbcli too.

Great!