Skip to content
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

Retry strategy to attach listener to activity #7

Closed
wants to merge 2 commits into from

Conversation

kristfal
Copy link

Not the most graceful attempt to fix #5

@kristfal
Copy link
Author

Not tested as we can't really manage to trigger the race condition.

We'll put this in production this week, will let you know how it fares.

@simonhoss
Copy link
Owner

Awesome. Thank you. I wait for your feedback then!

@kristfal
Copy link
Author

kristfal commented Feb 9, 2019

@simonhoss no crashes in prod, nor any complaints about locked UIs any longer 🍏

@kristfal
Copy link
Author

Got to backtrack on this one. We managed to reproduce it still. I’ll look into another fix.

@kristfal kristfal closed this Feb 16, 2019
@simonhoss
Copy link
Owner

Ok. Thank you for your investment!

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

Successfully merging this pull request may close these issues.

java.lang.RuntimeException: TouchThroughTouchHandlerInterface was not set on app activity
2 participants