Merge lp://qastaging/~dude-from-by/openlp/my-ios-remote into lp://qastaging/~danielborges93/openlp/ios-remote
Proposed by
andrei
Status: | Needs review |
---|---|
Proposed branch: | lp://qastaging/~dude-from-by/openlp/my-ios-remote |
Merge into: | lp://qastaging/~danielborges93/openlp/ios-remote |
Diff against target: |
588 lines (+295/-48) 10 files modified
OLP RemoteTests/SettingsViewModelTests.swift (+77/-0) Podfile (+1/-0) Podfile.lock (+4/-1) Remote.xcodeproj/project.pbxproj (+14/-0) Remote/AppDelegate.swift (+11/-1) Remote/Classes/Controller/Settings/SettingsControllerPresenter.swift (+2/-4) Remote/Classes/Controller/Settings/SettingsViewController.swift (+24/-40) Remote/Classes/Controller/Settings/SettingsViewModel.swift (+73/-0) Remote/Classes/Network/Service.swift (+4/-2) Remote/Classes/Util/CountlyClientSideUserPaths.swift (+85/-0) |
To merge this branch: | bzr merge lp://qastaging/~dude-from-by/openlp/my-ios-remote |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Daniel Borges | Needs Fixing | ||
Review via email:
|
This proposal supersedes a proposal from 2017-05-16.
Description of the change
Moved SettingsViewCon
Added Countly and events
To post a comment you must log in.
Events are described in CountlyClientSi deUserPaths. swift
first app launch start with no server ip
settings screen was opened without server ip filled
settings screen was closed with server ip filled
received server response - setup complete