Skip to content

Conversation

@kvld
Copy link
Contributor

@kvld kvld commented Nov 29, 2018

Release 1.73 milestone

Main & Common

  • Added from_mobile_app param #414
  • Removed AppsFlyer #417
  • Removed "Join" title A/B #418
  • Fixed deeplink fallback #419
  • Achievements popup A/B #420
  • Subscribe to notifications on launch A/B #416

ivan-magda and others added 5 commits November 28, 2018 15:16
* Add from mobile query parameter

Add from mobile query parameter to the DeepLinkRouter's url.

* Fix

* Create Extensions group with folder

* Fix

* Remove appendFromMobileQueryParameter()

* appendingQueryParameters
* removed join course string ab

* removed test localizations
@kvld kvld requested a review from SunInJuly November 29, 2018 20:04
* Create group with folder

* Split test

* Fix

* Respect current group

* Do not request authorization before onboarding

* Request authorization after onboarding

* Request authorization for all users if they participants

* Make decisions based on current group

* Refactor rename isParticipant -> shouldShowOnFirstLaunch

* Suggest streak
ivan-magda and others added 4 commits December 3, 2018 13:01
* Add split test

* Load progress

* Add analytics

* Move source

* Refactor rename

* Use only AchievementPopupAlertManager for presenting

* Respect current group

* Refactor rename isParticipant -> shouldShowAchievementPopup

* Optional progress for AchievementViewData

* Fix after merge conflict setSplitTestGroup

* Fix foreground notification presentation

Completes only with the sound when is in AB test group and with alert, sound when in control group

* Fix remove Recovered References group

* Fix alert not showing when in foreground for test group

We should always call the completion hadnler with provided .alert and .sound options, because only after pressing on the notification
notifications service will handleRemoteNotification.
* Fix custom alert presentation decision flow

Present custom permission alert if delegate allowed, otherwise request authorization.

* Format code
@kvld kvld merged commit 173df78 into master Dec 3, 2018
@kvld kvld deleted the release/1.73 branch January 29, 2019 11:35
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.

5 participants