irufus / gdax-java

Java based wrapper for Coinbase Pro (Formerly known as GDAX API and Coinbase Exchange API)
MIT License
177 stars 131 forks source link

Syntax error on WebsocketFeed #63

Closed jklappenbach closed 4 years ago

jklappenbach commented 4 years ago

this.objectMapper = objectMapper(new JavaTimeModule());

should probably be:

this.objectMapper = objectMapper.registerModule(new JavaTimeModule());

Otherwise, I get the following error when executing build tasks:

.../gdax-java/src/main/java/com/coinbase/exchange/api/websocketfeed/WebsocketFeed.java:58: error: cannot find symbol this.objectMapper = objectMapper(new JavaTimeModule()); ^ symbol: method objectMapper(JavaTimeModule) location: class WebsocketFeed Note: Some input files use or override a deprecated API. Note: Recompile with -Xlint:deprecation for details. Note: Some input files use unchecked or unsafe operations. Note: Recompile with -Xlint:unchecked for details. 1 error

FAILURE: Build failed with an exception.

irufus commented 4 years ago

~It looks like the JVM version isn't specified as a source/target build in the gradle file. This would cause an issue if your project isn't set to Java 11. I will take a look at this.~

Apparently I was looking at the wrong issue as I was having issues compiling as well. I've made the corrections in my branch which it should be merged into master soon

irufus commented 4 years ago

Fixed syntax issue