orta / ARAnalytics

Simplify your iOS/Mac analytics
MIT License
1.84k stars 217 forks source link

[WIP] Add breadcrumb logging to HockeyApp #155

Closed alloy closed 9 years ago

alloy commented 9 years ago

This will log events and ARLog to the Apple System Logging facility and use those to then add context to HockeyApp crash reports.

alloy commented 9 years ago

screen shot 2015-05-08 at 20 23 02

orta commented 9 years ago

Best to add a responds to selector for appProcessIdentifier given that we have no guarantee if which hockey version someone is using with ARAnalytics

alloy commented 9 years ago

Oh, I was wanting to specify a minimum required version of HockeySDK. So you prefer the agnostic version that checks for the method existing?

orta commented 9 years ago

I'd prefer that, I've had to downgrade hockey's before

alloy commented 9 years ago

@orta Did done :+1:

orta commented 9 years ago

I'm pleased. This is basically mergable, let's make the README say "in a future versions of Hockey", and I think there's a CHANGELOG on ARAnalytics, let's get it in there.

alloy commented 9 years ago

@orta I’ve asked the hockey people if it’s something they’d merge and if they can give an estimate of the version number. Without a version that README entry is not that useful, so let’s give it a day :)

orta commented 9 years ago

rockin

orta commented 9 years ago

This doesn't do a perform selector, it won't compile if you're not using your fork:

screen shot 2015-05-11 at 14 54 12

alloy commented 9 years ago

@orta Doh.

alloy commented 9 years ago

https://github.com/bitstadium/HockeySDK-iOS/pull/153

This can be merged :+1:

orta commented 9 years ago

Yeah, cool, whoever sees it green can merger.

alloy commented 9 years ago

I can’t merge :)

orta commented 9 years ago

hah, well, then all the power is mine

alloy commented 9 years ago

Oh, I forgot CHANGELOG