fix: Convert assetId to number

[rackstar]

Context

open-cover asked why asset.id was string and if we would support input of assetId as string.

looking at our smart-contracts it seems we take in assetId as a number Pool.getAsset(uint assetId). So maybe we should enforce that all assetId should be in number format.

Although this is technically a breaking change, since open-cover is the only consumer for now and this was only released earlier today maybe we can fix it otherwise we'll be stuck with this inconsistency.

Changes proposed in this pull request

• change asset.id to number
• update openapi docs

Test plan

• update test expectations for asset.id to be a number

Checklist

• [x] Rebased the base branch
• [x] Attached corresponding Github issue
• [x] Prefixed the name with the type of change (i.e. feat, chore, test)
• [x] Performed a self-review of my own code
• [x] Followed the style guidelines of this project
• [x] Made corresponding changes to the documentation
• [x] Didn't generate new warnings
• [x] Didn't generate failures on existing tests
• [x] Added tests that prove my fix is effective or that my feature works

Review

When reviewing a PR, please indicate intention in comments using the following emojis:

• :cake: = Nice to have but not essential.
• :bulb: = Suggestion or a comment based on personal opinion
• :hammer: = I believe this should be changed.
• :thinking: = I don’t understand something, do you mind giving me more context?
• :rocket: = Feedback

[roxdanila]

does our UI consume this too?

[rackstar]

does our UI consume this too?

I don't think so. I seem to remember @mixplore mentioning FE has 6 hardcoded for USDC

[rackstar]

does our UI consume this too?

I don't think so. I seem to remember @mixplore mentioning FE has 6 hardcoded for USDC

@Gideonnn confirmed the new asset object is not consumed in the FE