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

[TIMOB-23901] Undo changes for running on main thread #8406

Closed
wants to merge 2 commits into from

Conversation

ashcoding
Copy link
Contributor

@ashcoding ashcoding commented Sep 21, 2016

JIRA: https://jira.appcelerator.org/browse/TIMOB-23901

This PR aims to undo changes done by #7332 for https://jira.appcelerator.org/browse/TIMOB-19765.
Please do not merge yet. This PR is up for discussion first.

@ashcoding
Copy link
Contributor Author

@sgtcoolguy and @garymathews Please take a look.

@sgtcoolguy
Copy link
Contributor

@ashcoding Is there a reason we need to roll it all back to not even allow running on main thread? I would think we'd want to still allow using main thread and simply fix the bug that stems from the combination of analytics, being on the main thread, and restarting the app.

@garymathews
Copy link
Contributor

@ashcoding
Copy link
Contributor Author

Closing this PR.

@ashcoding ashcoding closed this Sep 22, 2016
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.

None yet

3 participants