Skip to content

Add decimal volume type and update trades, aggs and snapshot to use it.#564

Closed
dhiaayachi wants to merge 2 commits intomasterfrom
dhia/BACKEND-1005/add-factional-volume
Closed

Add decimal volume type and update trades, aggs and snapshot to use it.#564
dhiaayachi wants to merge 2 commits intomasterfrom
dhia/BACKEND-1005/add-factional-volume

Conversation

@dhiaayachi
Copy link

Add support to decimal_volume, the volume including fractional shares volume.

@justinpolygon
Copy link
Collaborator

Hi @dhiaayachi, this is likely not needed in that I'm going to auto generate the client. I'm just flipping this over and will release a breaking change. So, you can just assign this linear task to me and I'll resolve it.

@dhiaayachi
Copy link
Author

Hi @dhiaayachi, this is likely not needed in that I'm going to auto generate the client. I'm just flipping this over and will release a breaking change. So, you can just assign this linear task to me and I'll resolve it.

Thank you @justinpolygon for taking this! I will assign the ticket to you, just a heads up about the new field being a string. That was decided on purpose to allow conveying as much precision in the float as possible but we were thinking to provide some conveniency functions to do float conversion in the different libraries.

@dhiaayachi dhiaayachi closed this Feb 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants