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

Add RMF messageShown attribute #3062

Merged
merged 4 commits into from
Aug 4, 2024
Merged

Conversation

samsymons
Copy link
Collaborator

@samsymons samsymons commented Aug 2, 2024

Task/Issue URL: https://app.asana.com/0/1199333091098016/1207959177057413/f
Tech Design URL:
CC:

Description:

This PR adds support for the RMF messageShown attribute.

Steps to test this PR:

  1. https://app.asana.com/0/1199333091098016/1207959177057418/f

Definition of Done:


Internal references:

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

Copy link
Collaborator

@ayoy ayoy left a comment

Choose a reason for hiding this comment

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

LGTM

samsymons added a commit to duckduckgo/BrowserServicesKit that referenced this pull request Aug 4, 2024
Required:

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

Description:

This PR adds support for the RMF messageShown attribute, which will match against messages which have simply been displayed to the user, without any interaction required.
# By Dax the Duck (2) and others
# Via Alexey Martemyanov (2) and GitHub (2)
* main:
  Update autoconsent to v10.13.0 (#3063)
  [DuckPlayer] 1. Move Extension to BSK and add Init Updates (#3055)
  Bump version to 1.100.0 (238)
  Bump version to 1.100.0 (237)
  Improve sorting logic and related test (#3064)
  VPN Waitlist and Debug Menu code cleanup. (#3059)
  restore CrashLogMessageExtractor (#3011)

# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
#	DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
@samsymons samsymons merged commit 149b303 into main Aug 4, 2024
18 checks passed
@samsymons samsymons deleted the sam/add-rmf-message-shown-attribute branch August 4, 2024 20:00
samsymons added a commit that referenced this pull request Aug 4, 2024
# By Dax the Duck (8) and others
# Via GitHub (5) and Alexey Martemyanov (2)
* main: (44 commits)
  Add RMF `messageShown` attribute (#3062)
  Update autoconsent to v10.13.0 (#3063)
  [DuckPlayer] 1. Move Extension to BSK and add Init Updates (#3055)
  Bump version to 1.100.0 (238)
  Bump version to 1.100.0 (237)
  Improve sorting logic and related test (#3064)
  VPN Waitlist and Debug Menu code cleanup. (#3059)
  restore CrashLogMessageExtractor (#3011)
  xattr command removed from the restarting script to avoid the macOS notification (#3057)
  Duck Player - Improved Settings + Ship Review feedback (#2981)
  Fix flaky TabContentTests (#3060)
  PIR Database Migrations: Remove Temporary Internal Time-Based Feature Flag (#3054)
  Bump version to 1.100.0 (236)
  Remove unused messaging system (#3047)
  Automatically add Asana task to macOS App Board (#3053)
  Disable password generation for burner windows (#3024)
  PIR: Improve sorting logic and related test (#3049)
  Remove @mainactor from test case class definitions (#3048)
  Fix autoplay FF behavior (#3032)
  Fix ghost InfoPlist.xcstrings (#3044)
  ...

# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
#	DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
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