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

fix: Abort execution of completion block after dynamic config refresh completes if self is now nil (#288) #289

Merged

Conversation

jaltreuter
Copy link
Contributor

Hopefully this meets your style guidelines; it's been a while since I coded Objective-C.

@jaltreuter jaltreuter force-pushed the bugs/dynamic_config_dealloc_crash branch from 834f5b1 to 93a7094 Compare October 5, 2020 21:22
@jooohhn
Copy link
Contributor

jooohhn commented Oct 6, 2020

Thanks for the PR! We're working on a new release workflow and will merge this once it's up

@jooohhn jooohhn changed the title Abort execution of completion block after dynamic config refresh completes if self is now nil (#288) fix: Abort execution of completion block after dynamic config refresh completes if self is now nil (#288) Oct 7, 2020
@jooohhn jooohhn merged commit 9dc896d into amplitude:master Oct 7, 2020
github-actions bot pushed a commit that referenced this pull request Oct 20, 2020
## [7.1.1](v7.1.0...v7.1.1) (2020-10-20)

### Bug Fixes

* **buildsettings:** Remove override for GCC_WARN_INHIBIT_ALL_WARNINGS ([#302](#302)) ([0e55297](0e55297))
* **deprecation warnings:** Fix deprecation warnings ([#301](#301)) ([e7b0e6e](e7b0e6e)), closes [/github.com//issues/250#issuecomment-655224554](https://github.com//github.com/amplitude/Amplitude-iOS/issues/250/issues/issuecomment-655224554)
* **deprecations:** Use DEPRECATED_MSG_ATTRIBUTE instead of notes ([#305](#305)) ([f501c6c](f501c6c))
* nil dynamic config refresh crash ([#288](#288)) ([#289](#289)) ([9dc896d](9dc896d))
* Swift UserId and DeviceId setter ([#299](#299)) ([b7c0f90](b7c0f90))
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.

2 participants