-
Notifications
You must be signed in to change notification settings - Fork 0
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
GOVUKAPP-630 Clear visited Items #113
base: develop
Are you sure you want to change the base?
Conversation
…ov/govuk-mobile-ios-app into feature/recent-activity-clear-all
Quality Gate passedIssues Measures |
@@ -76,6 +76,13 @@ extension AppEvent { | |||
) | |||
} | |||
|
|||
static func clearRecentActivity() -> AppEvent { | |||
.init( | |||
name: "Cleared_recent_activity", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This doesn't seem to match the other name formats for AppEvent?
"recentActivityNavigationTitle" | ||
) | ||
func deleteActivities() { | ||
for item in model.todaysActivites { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be done in the ActivityService -> ActivityRepository
It should also really be done on the background context.
} | ||
model.todaysActivites.removeAll() | ||
model.currentMonthActivities.removeAll() | ||
model.recentMonthActivities.removeAll() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What would happen here if a previous save failed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like snapshots are potentially wrong here... plenty of snapshots here that shouldn't have changed
This pr adds clear all functionality to the recent activity view