C.S.S Bump (Via BSK) #3346
C.S.S Bump (Via BSK) #3346
Conversation
There was a problem hiding this comment.
LGTM.
I dind't see this "Adds subfeature flag" being used in the actual code, only added to BSK, is this expected?
Also, I found an issue that looks like an edge case. If you open settings and quickly tap back, it will take you back to the Duck Player settings again. Please see the attached video.
I don't think this is caused by this PR; I suggest filing it as a separate task.
settings.mp4
No. There was a missing commit. Just pushed.
Correct. Not caused by this PR. This is how deeplinks in settings are currently working. There's not much we can do to fix that for now, but I'll keep track of that. |
# By Alessandro Boron (3) and others # Via GitHub * main: BSK - Add feature flag for SKAN API (#3356) Remove checking for negative attribution case (#3355) C.S.S Bump (Via BSK) (#3346) Update Onboarding gradients (#3350) Alessandro/onboarding copy and private search options (#3349) Onboarding Intro - Add choose address bar position (#3340) Fix PrivacyDashboard appearance on entering foreground (#3345) Improve Data Store ID managing (#3335) # Conflicts: # DuckDuckGo.xcodeproj/project.pbxproj # DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
* main: Ensure toast closures are called on the main thread (#3347) Update survey builder OS version (#3348) BSK - Add feature flag for SKAN API (#3356) Remove checking for negative attribution case (#3355) C.S.S Bump (Via BSK) (#3346) Update Onboarding gradients (#3350) Alessandro/onboarding copy and private search options (#3349) Onboarding Intro - Add choose address bar position (#3340)
# By Alessandro Boron (7) and others # Via Bartek Waresiak (1) and others * main: (31 commits) Add Marketplace Postback handling (#3357) SKAD4 crash fix (#3361) Enroll all internal users in experiment && Update BSK (#3359) update for macOS: visited links (#3353) Ensure toast closures are called on the main thread (#3347) Update survey builder OS version (#3348) BSK - Add feature flag for SKAN API (#3356) Remove checking for negative attribution case (#3355) C.S.S Bump (Via BSK) (#3346) Update Onboarding gradients (#3350) Alessandro/onboarding copy and private search options (#3349) Onboarding Intro - Add choose address bar position (#3340) Fix PrivacyDashboard appearance on entering foreground (#3345) Improve Data Store ID managing (#3335) Alessandro/onboarding choose app icon (#3330) Fix Localizable strings (#3341) Move lazy var access to the MainActor (#3333) Release 7.137.0-2 (#3344) Fix privacy icon glitch (#3343) BSK bump for macOS password import promotion flow (#3332) ... # Conflicts: # DuckDuckGo.xcodeproj/project.pbxproj # DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
Task/Issue URL: https://app.asana.com/0/1204099484721401/1208271424696738/f
BSK PR: duckduckgo/BrowserServicesKit#983
Description:
Steps to test this PR: