java 1.7 support?

[chrisptang]

Hi, it seems like current package only supports java 1.8+, while some of our exists projects are using 1.7 or older. It requires a lot of extra works to upgrade all our projects to java 1.8. Would you plan to support java1.7 in the future?

[locspin]

Seconded. Only supporting Java 1.8+ requires similar changes in our
ecosystem. We are manually calling the API until that is resolved, which is
a pain.

On Nov 15, 2016 7:41 PM, "Peng Tang" notifications@github.com wrote:

Hi, it seems like current package only supports java 1.8+, while some of
our exists projects are using 1.7 or older. It requires a lot of extra
works to upgrade all our projects to java 1.8. Would you plan to support
java1.7 in the future?

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#61, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AVJn0vZWAsiLflBFrK_wHgj4Q75Z0ngyks5q-nt8gaJpZM4KzUgo
.

[chrisptang]

Hi @locspin
I just build a jdk1.7 version: https://github.com/chrisptang/facebook-java-ads-sdk/tree/master/target

Main changes:

• Use org.apache.commons.lang.StringUtils.join instead of java8 String.join
• Fix this NullPointerException bug: NullPointerException throws while trying to access /me/businesses #60

Enjoy.

[lintaocui]

Hi @chrisptang @locspin , thank you for reporting the issue. Support Java 1.7 sounds reasonable, given there are already two asks for it. We are keeping this issue tracked.

[JiamingFB]

Java 7 is now supported in latest release

[chrisptang]

Sounds great.