polygon-io / client-jvm

The official JVM client library SDK, written in Kotlin, for accessing the Polygon REST and WebSocket API.
55 stars 34 forks source link

Sort Parameter Missing From Aggregates (Bars) #143

Closed noah-hein closed 11 months ago

noah-hein commented 11 months ago

While using the API I noticed the sort field is missing from the Aggregate (Bars) parameters. Added in the missing parameter to the Aggregate.kt file and made sure it was placed as a URL parameter. This was a very simple few lines modified. If I missed any test cases or samples please let me know.

Screenshot of the official documentation showcasing the sort parameter Screenshot 2023-12-03 062906

Screenshot of the decompiled Kotlin file for the current version missing the parameter Screenshot 2023-12-03 063131

justinpolygon commented 11 months ago

Hey @noah-hein, thanks very much for the PR that adds the sort param, updating the examples, and the spec too. What a great PR! Cheers.