getditto / DittoAndroidTools

Diagnostic and Debugging Tools for Ditto Android SDK
MIT License
6 stars 1 forks source link

Update grade dependencies and fix for some crashes #120

Closed rajramsaroop closed 3 weeks ago

rajramsaroop commented 4 weeks ago

Now that we've published all tools we should be pulling in the dependencies using gradle as opposed to specifying the local project(s).

I also noticed a few crashes when integrating with apps outside the project. Export logs and Heartbeat for example. In it's current state you can reproduce those crashes by pulling in Tools Viewer 2.1.0 (latest published) and trying to use Export Logs or Heartbeat. I tested this with the demo app, chat app and Android PoS demo [of which I am working on locally].

To see it fixed, run ./gradlew publishToMavenLocal for tools, then pull in the local tools version (SNAPSHOT).

Export Logs crashed due to a null value being returned from a predicate. Heartbeat crashed because gradle couldn't figure out which class to use for DittoCollection.upsert().

Once these fixes are in, I'd like to publish a 2.1.1. version.

rajramsaroop commented 4 weeks ago

Also make sure to update the README - now the mavenLocal dependency step is not required any more!

It's still required for external projects