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 publisher, add option to override internal user state in debug #422

Merged
merged 2 commits into from
Jul 19, 2023

Conversation

bwaresiak
Copy link
Collaborator

Please review the release process for BrowserServicesKit here.

Required:

Task/Issue URL: https://app.asana.com/0/414709148257752/1205091275850283/f
iOS PR:
macOS PR:
What kind of version bump will this require?: Major/Minor/Patch

Optional:

Tech Design URL:
CC:

Description:

Allow to set internal user state in debug.

Steps to test this PR:

Navigate to settings, tests if this works as expected.

OS Testing:

  • iOS 14
  • iOS 15
  • iOS 16
  • macOS 10.15
  • macOS 11
  • macOS 12

Internal references:

Software Engineering Expectations
Technical Design Template

@brindy brindy self-requested a review July 19, 2023 09:01
Copy link
Contributor

@brindy brindy left a comment

Choose a reason for hiding this comment

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

LGTM, one super minor nit

@bwaresiak bwaresiak merged commit 687d881 into main Jul 19, 2023
@bwaresiak bwaresiak deleted the bartek/internal-user-toggle branch July 19, 2023 10:00
samsymons added a commit that referenced this pull request Jul 20, 2023
* main:
  Fix publisher, add option to override internal user state in debug  (#422)
  Removes unused code and makes some other code platform specific (#419)
  Optimize first load of Content Rule Lists (#406)
  Automatically add the PR link to the Asana task (#311)
  NetP: Move bundle prefix to x-platform code (#418)
afterxleep pushed a commit that referenced this pull request Aug 2, 2023
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