-
Notifications
You must be signed in to change notification settings - Fork 14
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
DBP remote messaging #1997
DBP remote messaging #1997
Conversation
# By Dominik Kapusta (2) and others # Via GitHub * main: Make sure when we set custom config url, we don't expect etag in return (#1994) Add PixelKit source parameter (#1989) Fix internal user toggling (#2000) Show alert and display warning icon in Sync Settings when data syncing is disabled (#1996) DBP: Integrate subscription account authentication to DBP (#1995) Improve bookmarks html reader (#1986) Add Sync feature flags (#1992) Add daily stats pixel (#1993) Do not reload DBP tab when switching to it (#1942) Fix: external application requests via redirect URLs shows wrong origin. (#1900) # Conflicts: # DuckDuckGo/Statistics/PixelEvent.swift
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.
Did the first run thru, found some small issues (most of them are nits that you can ignore).
DuckDuckGo/DBP/RemoteMessaging/DataBrokerProtectionRemoteMessaging.swift
Outdated
Show resolved
Hide resolved
DuckDuckGo/DBP/RemoteMessaging/DataBrokerProtectionRemoteMessaging.swift
Outdated
Show resolved
Hide resolved
# By Dax the Duck (6) and others # Via Fernando Bunn (2) and others * main: (36 commits) Send VPN system extension crashes to Sentry (#2002) Update login item failure pixel (#2024) fix turn off sync error message (#2025) DBP: macOS - Scheduler Progress Notifications (#2023) DBP: Implement sign-out flow for DBP (#2009) Switch CI to Xcode 15.1 (#2022) fix(duckplayer): bump CSS for duckplayer nav loop fix (#1982) Bump Submodules/privacy-reference-tests from `a3acc21` to `6b7ad1e` (#2006) Bump version to 1.69.0 (99) DBP: Respect foreign constraints when deleting all user's data (#2014) DBP: Respect foreign constraints when deleting all user's data (#2014) Don't force reload tab when restoring state (#2016) Bump version to 1.69.0 (98) Update embedded files Update NetP launch agent logic to include build number (#2015) Always use 'sandbox' Application Support directory for Favicons Fetcher (#2013) Allow calculations in the address bar (#2012) Fix clickable area for buttons in 'Sync with Another Device' view (#2011) Fix for empty autofill state displayed on top of existing password items (#1998) Update error messages (#1999) ... # Conflicts: # DuckDuckGo.xcodeproj/project.pbxproj # DuckDuckGo/Statistics/PixelEvent.swift
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.
LGTM, I tested the case in the description and it works as intended.
ps. I've noticed that this changes some NetP related code but I haven't tested those use cases, the code looks good though.
I've added some small comments.
Nice work :)
# By Dominik Kapusta (3) and others # Via GitHub * main: Move Preferences views to SwiftUIExtension package (#2035) Add API to obtain reason why given feature is disabled (#2030) Removes Home-button temporary pixels (#2027) add rollout banner (#2031) Update Enter Code view in Sync Settings (#2029) Use checkboxes instead of switches for Sync Settings options (#2026) # Conflicts: # DuckDuckGo/Statistics/PixelEvent.swift # DuckDuckGo/Statistics/PixelParameters.swift
* main: (24 commits) Autofill management UI and subdomain matching (#2018) Update BSK with autofill 10.0.3 (#2051) display "you can delete passwords file" for Safari (#2050) Bump version to 1.70.0 (101) Update embedded files Fix permission request (#2046) update banner colour (#2044) update banner colour (#2044) Force reload on DBP notifications (#2028) update copy (#2042) Suspend Sync when updating device name (#2037) update copy (#2042) Update Enter Code view in Sync Account Recovery (#2038) Suspend Sync when updating device name (#2037) Extend hover area for address bar's bookmark button over bookmarks bar and tab bar (#2020) Bump version to 1.70.0 (100) Update embedded files DBP remote messaging (#1997) Move Preferences views to SwiftUIExtension package (#2035) Add API to obtain reason why given feature is disabled (#2030) ...
Task/Issue URL: https://app.asana.com/0/0/1206209106966759/f
Tech Design URL:
CC:
Description:
This PR adds support for showing remote messages to DBP users.
Steps to test this PR:
WaitlistActivationDateStore
and editdaysSinceActivation()
to return10
(or some higher value)Internal references:
Pull Request Review Checklist
Software Engineering Expectations
Technical Design Template
Pull Request Documentation