Skip to content

Commit 2643c11

Browse files
Merge branch 'trunk' into issue/6969_filter_for_email_followers
2 parents c4a1f53 + ed33989 commit 2643c11

File tree

3 files changed

+5
-4
lines changed

3 files changed

+5
-4
lines changed

RELEASE-NOTES.txt

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,8 @@
33
* [**] Some of the screens of the app has a new, fresh and more modern visual, including the initial one: My Site. [#17812]
44
* [**] Notifications: added a button to mark all notifications in the selected filter as read. [#17840]
55
* [**] People: you can now manage Email Followers on the People section! [#17854]
6+
* [*] Stats: fix navigation between Stats tab. [#17856]
7+
* [*] Quick Start: Fixed a bug where a user logging in via a self-hosted site not connected to Jetpack would see Quick Start when selecting "No thanks" on the Quick Start prompt. [#17855]
68

79
19.1
810
-----

WordPress/Classes/ViewRelated/Blog/QuickStartSettings.swift

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -35,9 +35,7 @@ final class QuickStartSettings {
3535
}
3636

3737
private func promptWasDismissedKey(for blog: Blog) -> String? {
38-
guard let siteID = blog.dotComID?.intValue else {
39-
return nil
40-
}
38+
let siteID = blog.dotComID?.intValue ?? 0
4139
return "QuickStartPromptWasDismissed-\(siteID)"
4240
}
4341

WordPress/Classes/ViewRelated/Stats/SiteStatsDashboardViewController.swift

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -194,10 +194,11 @@ private extension SiteStatsDashboardViewController {
194194
let selectedPeriodChanged = currentSelectedPeriod != oldSelectedPeriod
195195
let previousSelectedPeriodWasInsights = oldSelectedPeriod == .insights
196196
let pageViewControllerIsEmpty = pageViewController?.viewControllers?.isEmpty ?? true
197+
let isGrowAudienceShowingOnInsights = insightsTableViewController.isGrowAudienceShowing
197198

198199
switch currentSelectedPeriod {
199200
case .insights:
200-
if selectedPeriodChanged || pageViewControllerIsEmpty {
201+
if selectedPeriodChanged || pageViewControllerIsEmpty || isGrowAudienceShowingOnInsights {
201202
pageViewController?.setViewControllers([insightsTableViewController],
202203
direction: .forward,
203204
animated: false)

0 commit comments

Comments
 (0)