krazkidd / kdeck

Desktop client for Kalshi event trading platform
GNU General Public License v3.0
1 stars 0 forks source link

Refactor static `Api` class as namespace #36

Closed krazkidd closed 5 days ago

krazkidd commented 1 week ago

Big static classes are awkward (and against best practice). Currently, we do not have the option to refactor to a namespace because we store API responses in the static class (as we are still prototyping UI functionality). This will likely have to change in order to implement #17 cleanly if not before. When we do that, we can convert the static class to free functions in a namespace (if needed).

krazkidd commented 1 week ago

Some resources from my notes: