Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Syntax error on WebsocketFeed #63

Closed
jklappenbach opened this issue May 23, 2020 · 2 comments · Fixed by #65
Closed

Syntax error on WebsocketFeed #63

jklappenbach opened this issue May 23, 2020 · 2 comments · Fixed by #65
Assignees
Labels

Comments

@jklappenbach
Copy link

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 irufus self-assigned this May 24, 2020
@irufus
Copy link
Owner

irufus commented May 24, 2020

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 irufus linked a pull request May 25, 2020 that will close this issue
@irufus
Copy link
Owner

irufus commented May 25, 2020

Fixed syntax issue

@irufus irufus closed this as completed May 25, 2020
@irufus irufus added the bug label May 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants