fix: place order tx should use marketInfo and currentHeight from abacus payload

[aforaleka]

this should skip 2 round trip calls for getting market info and block height. abacus already has that info from fetched market info (*for active market) and polled/estimated block height

• latter could potentially increase risk of some errors related to GTB, but will monitor next week

  * @param marketInfo optional market information for calculating quantums and subticks.
  *        This can be constructed from Indexer API. If set to null, additional round
  *        trip to Indexer API will be made.
  * @param currentHeight Current block height. This can be obtained from ValidatorClient.
  *        If set to null, additional round trip to ValidatorClient will be made.

  https://github.com/dydxprotocol/v4-clients/blob/56d8c2deacffd97eb70343b2904ac94f4c91d242/v4-client-js/src/clients/composite-client.ts#L342-L346

before:

testing:

• place order should still work and succeed
• no extra call to perpetual market info
• abacus still poll validator height so that one is a little unclear, but that call shouldn't be a blocker now

[vercel[bot]]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
v4-staging	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Aug 23, 2024 7:31pm
v4-testnet	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Aug 23, 2024 7:31pm

[aforaleka]

i think abacus has had those fields for quite a while, we just missed them when migrating / when they were added @tyleroooo t_t