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

Fix of homepage history feed issue after burning #620

Merged
merged 1 commit into from
Jun 24, 2022

Conversation

tomasstrba
Copy link
Contributor

Task/Issue URL: https://app.asana.com/0/1177771139624306/1202472796284065/f

Description:
This PR fixes a privacy issue of homepage history feed. In few cases, the feed was not refreshed after burning and contained burned domains

Steps to test this PR:

  1. Open a new window
  2. Visit wikipedia.org
  3. Open a new tab and visit www.instagram.com
  4. Open another tab and make sure instagram.com is part of the homepage history feed
  5. Click on the fire button, choose burn window, select instagram.com only and confirm clearing
  6. Make sure instagram.com is not part of the homepage feed

Internal references:

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

@tomasstrba tomasstrba changed the title Update homepage history feed after burning Update homepage history feed after each burning Jun 20, 2022
@tomasstrba tomasstrba changed the title Update homepage history feed after each burning Fix of homepage history feed issue after burning Jun 20, 2022
@Bunn Bunn self-assigned this Jun 21, 2022
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

@Bunn Bunn assigned tomasstrba and unassigned Bunn Jun 24, 2022
@ayoy ayoy merged commit 3ed10c4 into develop Jun 24, 2022
@ayoy ayoy deleted the tom/homepage-privacy-issue branch June 24, 2022 10:27
samsymons added a commit that referenced this pull request Jun 29, 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)
samsymons added a commit that referenced this pull request Jul 4, 2022
# By Dominik Kapusta (5) and others
# Via Dominik Kapusta (1) and GitHub (1)
* develop:
  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)
  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

# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
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