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

Remove email waitlist #615

Merged
merged 9 commits into from
Aug 23, 2022
Merged

Remove email waitlist #615

merged 9 commits into from
Aug 23, 2022

Conversation

samsymons
Copy link
Collaborator

Task/Issue URL: https://app.asana.com/0/1199230911884351/1202144584700631/f
Tech Design URL:
CC:

Description:

This PR removes the email waitlist feature. As far as the macOS app is concerned, this change is pretty simple, since it never included any of the waitlist code. All that's done here is update the email URL.

Steps to test this PR:

  1. Run this branch, go to the Options menu and select Email Protection -> Enable Email Protection
  2. You should be taken to a new tab with the email protection URL. This might show weird state, the frontend changes aren't done yet - this branch is pointing to quackdev for testing purposes also.
  3. Go to duckduckgo.com/email/login and sign in
  4. Check that the Options menu's Email Protection menu is now accurate, you should see the option to generate an alias or sign out
  5. Go to a website that has an email form (nextdraft.com) and check that Dax shows up
  6. From the web dashboard, click Sign Out at the bottom of the page
  7. Verify that you are signed out and that the Options menu's Email Protection menu is accurate
  8. Go to a website that has an email form (nextdraft.com) and check that Dax does not show up any more

Testing checklist:

  • Test with Release configuration
  • Test proper deallocation of tabs
  • Make sure committed submodule changes are desired

Internal references:

Software Engineering Expectations
Technical Design Template
When ready for review, remember to post the PR in MM

@samsymons samsymons changed the title [Draft] Remove email waitlist Remove email waitlist Jun 15, 2022
@amddg44 amddg44 self-assigned this Jun 16, 2022
Copy link
Contributor

@amddg44 amddg44 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 👍

@amddg44 amddg44 assigned samsymons and unassigned amddg44 Jun 16, 2022
* develop:
  Update sparkle-sandbox.sh README to highlight proper usage
  Replace altool with notarytool in release build script (#623)
  Update embedded data
  Bump version to 0.26.4
  Update homepage history feed after burning (#620)
  Firefox data import improvements (#622)
  Set version to 0.26.3
  Microsoft logo for address bar animation (#618)
  Google spreadsheets login fix (#616)
# By Fernando Bunn (6) and others
# Via Chris Brind (3) and others
* develop:
  update files and bump release verison
  Check the image rule compilation error correctly (#635)
  Check that the alerting WKWebView is currently selected. (#637)
  Fix issue when unplugging external monitor (#629)
  Select tabs when browser is not focused (#632)
  Revert "Update BSK for iOS app group (#630)" (#634)
  Use updated BSK branch (#627)
  Version 0.26.5
  Embedded files updated
  Update BSK for iOS app group (#630)
  Recently Closed Menu Item (#617)
  Close dashboard when closing tab with cmd + w (#628)
  chore(deps): bump Autofill to 4.7.0 (#631)
  Handle legacy crash logs and use proper architecture in symbolicate.js (#621)
  Detect when Chrome login fails due to keychain prompt denial (#633)
  Correctly display unprotected sites on dashboard (#624)
  Update homefavicon (#625)

# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
# By Sam Symons (14) and others
# Via GitHub (1) and others
* develop: (45 commits)
  Revert "Prevent the app from quitting while the Fire button is active" (#690)
  Disable link preview. (#691)
  Fire Button quick wins (#677)
  Bookmarks phase 2 bug fixes (#686)
  Set version to 0.28.3.
  Only clear the selection if the selection needs to change. (#689)
  Set version to 0.28.2.
  Update embedded files for 0.28.2.
  Attempt to stop the new tab button from jumping around the UI (#685)
  Fix of threading issues related to history causing crashes  (#688)
  Add empty autofill isEnabled checks (to update BSK) (#683)
  Bump BSK to a version that uses updated GRDB (#647)
  Fix Save Credentials popover on macOS Ventura (#672)
  Update BSK to latest version + noop updates (#669)
  Update selectedTabViewModel only when it actually changes (#681)
  Update BSK to version 21.0.0 (#680)
  Always add the Autoconsent script (#666)
  Optimisation of history entry saving (#675)
  Use localized compare to sort autofill items (#674)
  0.28.1
  ...

# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
samsymons added a commit to duckduckgo/BrowserServicesKit that referenced this pull request Aug 18, 2022
Required:

Task/Issue URL: https://app.asana.com/0/0/1202144584700632/f
iOS PR: duckduckgo/iOS#1190
macOS PR: duckduckgo/macos-browser#615
What kind of version bump will this require?: Major

Description:

This PR removes the email waitlist. See the task for more details.
* develop:
  Update the global accent color. (#695)
  Version 0.28.4
  Embedded files updated
@samsymons
Copy link
Collaborator Author

Test failures are unrelated and also occur on develop, merging this.

@samsymons samsymons merged commit 3e19d53 into develop Aug 23, 2022
@samsymons samsymons deleted the sam/remove-email-waitlist branch August 23, 2022 23:03
tomasstrba pushed a commit that referenced this pull request Aug 24, 2022
Task/Issue URL: https://app.asana.com/0/1199230911884351/1202144584700631/f
Tech Design URL:
CC:

Description:

This PR removes the email waitlist feature. As far as the macOS app is concerned, this change is pretty simple, since it never included any of the waitlist code. All that's done here is update the email URL.
samsymons added a commit that referenced this pull request Aug 25, 2022
* develop:
  Version 0.28.5
  Embeded files updated
  Remove email waitlist (#615)
  update the menu once the favicons have loaded (#696)
  Remove email waitlist (#615)
  Update the global accent color. (#695)
  Version 0.28.4
  Embedded files updated
  Revert "Prevent the app from quitting while the Fire button is active" (#690)
  Disable link preview. (#691)
samsymons added a commit that referenced this pull request Aug 25, 2022
* develop:
  Version 0.28.5
  Embeded files updated
  Remove email waitlist (#615)
  update the menu once the favicons have loaded (#696)
  Remove email waitlist (#615)
  Update the global accent color. (#695)
  Version 0.28.4
  Embedded files updated
  Revert "Prevent the app from quitting while the Fire button is active" (#690)
  Disable link preview. (#691)
  Fire Button quick wins (#677)
  Set version to 0.28.3.
  Only clear the selection if the selection needs to change. (#689)
  Set version to 0.28.2.
  Update embedded files for 0.28.2.
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.

2 participants