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 Missing Enum to Freemium RMF Implementation #3631

Merged
merged 2 commits into from
Dec 5, 2024

Conversation

aataraxiaa
Copy link
Contributor

@aataraxiaa aataraxiaa commented Dec 4, 2024

Task/Issue URL: https://app.asana.com/0/1206488453854252/1208904081551825/f

Description: Bump BSK for missing Freemium RMF attribute

Optional E2E tests:

  • Run PIR E2E tests
    Check this to run the Personal Information Removal end to end tests. If updating CCF, or any PIR related code, tick this.

Steps to test this PR:

  1. Change the RMF debug URL in RemoteMessageClient to https://www.jsonblob.com/api/1313918156242345984
  2. In RemoteMessagingConfigMatcherProvider at line 168, pass in true for isCurrentFreemiumPIRUser
  3. Run the app
  4. Ensure you see a RMF message on the new tab page with a button that says "freemium"

Definition of Done:


Internal references:

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

@aataraxiaa aataraxiaa force-pushed the pete/complete-freemium-RMF-implementation branch from 712dfa1 to 5c7c50f Compare December 4, 2024 18:11
@aataraxiaa aataraxiaa marked this pull request as ready for review December 4, 2024 18:11
@aataraxiaa aataraxiaa requested a review from samsymons December 4, 2024 18:11
aataraxiaa added a commit to duckduckgo/BrowserServicesKit that referenced this pull request Dec 5, 2024
**Required**:

Task/Issue URL:
https://app.asana.com/0/1206488453854252/1208904081551825/f
iOS PR: duckduckgo/iOS#3673
macOS PR: duckduckgo/macos-browser#3631
What kind of version bump will this require?: Patch

**Description**: Adding missing Freemium RMF enum
@aataraxiaa aataraxiaa merged commit 170e7ca into main Dec 5, 2024
20 checks passed
@aataraxiaa aataraxiaa deleted the pete/complete-freemium-RMF-implementation branch December 5, 2024 10:33
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