Skip to content
This repository has been archived by the owner on Apr 4, 2023. It is now read-only.

addCrashLog missing in Countly.js #21

Closed
kwv opened this issue Aug 16, 2017 · 2 comments
Closed

addCrashLog missing in Countly.js #21

kwv opened this issue Aug 16, 2017 · 2 comments
Assignees

Comments

@kwv
Copy link

kwv commented Aug 16, 2017

The documentation refers to manually adding crash log:

// Send a custom crash log
Countly.addCrashLog("My crash log from JavaScript");

This API does not appear to be implemented (yet?) in Countly.js.

Is the intent of the API for application managed non-fatal errors (e.g. gracefully caught 500 responses)?

From tracing through the Android code, its not clear if this should be exposed. I didn't spend a lot of time, but I couldn't find what sends the CrashDetails generated in addCrashLog

Rather should logException be overloaded to take a string? That might make more sense to expose as addCrashLog in Countly.js?

@trinisofttechnologies
Copy link
Contributor

trinisofttechnologies commented Aug 16, 2017 via email

@trinisofttechnologies
Copy link
Contributor

@kwv

Sorry, I checked it, and seems like this method wasn't implemented.

I have added the method now, in this 550fefb commit.

Thanks

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants