8000 [ℹ️] NT-1143 Adding more info to TrackingWorker error logs by eoji · Pull Request #839 · kickstarter/android-oss · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

[ℹ️] NT-1143 Adding more info to TrackingWorker error logs #839

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

Merged
merged 2 commits into from
Apr 16, 2020

Conversation

eoji
Copy link
Contributor
@eoji eoji commented Apr 13, 2020

📲 What

Adding more info to TrackingWorker error logs.

🤔 Why

Tracking failures are a big mystery! Hopefully with more info in the logs, we can get down to the bottom of it.

🛠 How

  • Added the response.message and runAttemptCount to the error message that's logged when an event fails to track.

build.gradle

I had to add another repo for Apollo because it was causing the build to fail. More ammo for justification to get rid of it.

👀 See

Logs only.

📋 QA

We will see these failures in the Firebase console 🚀

Story 📖

NT-1143

Copy link
Contributor
@yatriks yatriks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🙏👍

@eoji eoji changed the title [ℹ️] Adding more info to TrackingWorker error logs [ℹ️] NT-1143 Adding more info to TrackingWorker error logs Apr 13, 2020
@eoji eoji merged commit 082cbc4 into master Apr 16, 2020
@eoji eoji deleted the tracking-error-logs branch April 16, 2020 17:40
@eoji eoji mentioned this pull request Apr 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0