Skip to content

release-6.6.0 fix #2 for GDTUploadCoordinator crashes #3557

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 3 commits into from
Aug 12, 2019

Conversation

mikehaney24
Copy link
Contributor

No description provided.

Fixes 3547, in theory. This code all goes away/changes with the sqlite refactor, so making it safe is probably sufficient for now. No one was able to reproduce locally, most likely because this is a lifecycle issue.
Copy link
Contributor

@maksymmalyhin maksymmalyhin left a comment

Choose a reason for hiding this comment

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

Would you like to bump the pod version in this PR as well?

@mikehaney24
Copy link
Contributor Author

Would you like to bump the pod version in this PR as well?

Yes.

GDTCCT has a dep on ~> 1.1, so this incremented version should get picked up.
@paulb777 paulb777 merged commit 518a44e into release-6.6.0 Aug 12, 2019
@paulb777 paulb777 deleted the mph-release-6.6.0-fix2 branch August 12, 2019 20:49
@firebase firebase locked and limited conversation to collaborators Oct 9, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants