Skip to content
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

Merged
merged 35 commits into from
Jan 10, 2024
Merged

DBP remote messaging #1997

merged 35 commits into from
Jan 10, 2024

Conversation

samsymons
Copy link
Collaborator

@samsymons samsymons commented Dec 21, 2023

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:

  1. Run this PR and check that no message appears
  2. Open WaitlistActivationDateStore and edit daysSinceActivation() to return 10 (or some higher value)
  3. Re-run the PR and check that a message appears
  4. If a message doesn't appear, wait a minute and try again - the remote message fetcher only fetches so often, but it uses a short timeout in debug mode

Internal references:

Pull Request Review Checklist
Software Engineering Expectations
Technical Design Template
Pull Request Documentation

Copy link
Contributor

github-actions bot commented Dec 21, 2023

Warnings
⚠️ PR has more than 500 lines of code changing. Consider splitting into smaller PRs if possible.

Generated by 🚫 dangerJS against 1cac5dd

# 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
Copy link
Collaborator

@Bunn Bunn left a 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).

@samsymons samsymons marked this pull request as ready for review December 23, 2023 01:47
samsymons and others added 12 commits December 22, 2023 18:45
# 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
Copy link
Collaborator

@Bunn Bunn left a 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
@samsymons samsymons merged commit 09113e8 into main Jan 10, 2024
14 checks passed
@samsymons samsymons deleted the sam/data-broker-remote-messaging branch January 10, 2024 03:29
samsymons added a commit that referenced this pull request Jan 11, 2024
* 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)
  ...
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.

3 participants