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

Change service name for NetP auth tokens. #479

Merged
merged 2 commits into from
Aug 25, 2023

Conversation

diegoreymendez
Copy link
Contributor

@diegoreymendez diegoreymendez commented Aug 25, 2023

Task/Issue URL: https://app.asana.com/0/0/1205354504536670/f
iOS PR: duckduckgo/iOS#1956
macOS PR: duckduckgo/macos-browser#1552
What kind of version bump will this require?: Minor

Description

Uses a default for the auth token service that does not depend on the app's bundle ID.

Testing

Test these changes in the iOS and macOS PRs.


Internal references:

Software Engineering Expectations
Technical Design Template

Copy link
Contributor

@samsymons samsymons left a comment

Choose a reason for hiding this comment

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

LGTM, tested via the macOS PR.

@diegoreymendez diegoreymendez merged commit 7221163 into main Aug 25, 2023
@diegoreymendez diegoreymendez deleted the diego/fix-netp-menu-not-showing branch August 25, 2023 18:22
diegoreymendez added a commit to duckduckgo/macos-browser that referenced this pull request Aug 25, 2023
Task/Issue URL: https://app.asana.com/0/0/1205354504536669/f
BSK PR: duckduckgo/BrowserServicesKit#479
iOS PR: duckduckgo/iOS#1956

## Description

An issue is causing the NetP menu to no longer open.
diegoreymendez added a commit to duckduckgo/iOS that referenced this pull request Aug 26, 2023
Task/Issue URL: https://app.asana.com/0/0/1205354504536671/f
BSK PR: duckduckgo/BrowserServicesKit#479
macOS PR: duckduckgo/macos-browser#1552

## Description

Updates to an upcoming version of BSK.
samsymons added a commit that referenced this pull request Aug 28, 2023
* main:
  Change service name for NetP auth tokens. (#479)
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