neopixeldev / hypixel-api-java

Asynchronous Java wrapper for the Hypixel API
MIT License
6 stars 2 forks source link

Make request factory return a `JSONHandler` #96

Closed unpluggedsam closed 1 year ago

unpluggedsam commented 1 year ago

In order to guarantee encapsulation of the JSONObject's be used throughout the wrapper the JSONHandler class was created. But, it wasn't instantiated as soon as the JSONObject was returned from the API. This PR will change that.

The benefit of this is proxy classes will not be instantiating their own JSONHandler's. Instead, it will automatically be returned from the RequestFactory.

Mqlvin commented 1 year ago

Everything is looking great, however there is a test class present.