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

support marketplace kit #2833

Merged
merged 7 commits into from
May 8, 2024
Merged

support marketplace kit #2833

merged 7 commits into from
May 8, 2024

Conversation

brindy
Copy link
Contributor

@brindy brindy commented May 8, 2024

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

Description:
Adds marketplace kit support. Installing alternative app stores doesn't appear to work with an adhoc build so need to test in the wild. Internal users only.

The final version should use our current link handling logic, this direct approach is just to validate it works.

Steps to test this PR:

  1. With an iOS 17.4 device or simulator with internal user flag enabled access https://chrisbrind.rocks/testing/ and click on the marketplace URL and you should get a toast. Check that tapping COPY copies the URL into the pasteboard.
  2. Try the above where any of the conditions are false (e.g. older version of iOS, internal user flag off, etc). Nothing should happen when clicking on the link.

Internal references:

Software Engineering Expectations
Technical Design Template

@@ -6,7 +6,6 @@ platform "ios"
type "appstore"
app_identifier ["com.duckduckgo.mobile.ios", "com.duckduckgo.mobile.ios.ShareExtension", "com.duckduckgo.mobile.ios.OpenAction2", "com.duckduckgo.mobile.ios.Widgets", "com.duckduckgo.mobile.ios.NetworkExtension"]
generate_apple_certs false
template_name "Default Web Browser iOS (Dist)"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This forces it to use the wrong profile. The entitlements are now driven by the app id configuration.

@brindy brindy requested review from a team, dus7, mallexxx, federicocappelli and aataraxiaa and removed request for a team, dus7, mallexxx and federicocappelli May 8, 2024 11:32
Copy link
Contributor

@aataraxiaa aataraxiaa left a comment

Choose a reason for hiding this comment

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

LGTM! 🚀

Minor clarification as discussed on MM: When clicking on the link with internal user disabled, something does happen, which is that the following toast is displayed

Screenshot 2024-05-08 at 13 19 55

@brindy brindy merged commit 5eda30c into main May 8, 2024
57 of 62 checks passed
@brindy brindy deleted the brindy/support-marketplace-kit branch May 8, 2024 12:32
samsymons added a commit that referenced this pull request May 9, 2024
* main:
  Upload logs on cancel (#2836)
  Add bookmarks to top hits suggestions on iOS (#2835)
  Fix Maestro reporting for e2e tests (#2713)
  support marketplace kit (#2833)
  Enable attribution flag (#2825)
  Release 7.119.0-0 (#2831)
  Fix subscriptions glitch in Settings (#2828)
  Delegate SecureVault KeyStore Events in All Scenarios  (#2827)
  Clears subscription cache on restore (#2823)
samsymons added a commit that referenced this pull request May 10, 2024
# By amddg44 (3) and others
# Via GitHub (1) and amddg44 (1)
* main:
  Enable gzip compression for Sync PATCH payloads (#2805)
  sync error notify (#2822)
  Wire up VPN error UI (#2826)
  Mergeback from release/7.119.0
  Release 7.119.0-1 (#2840)
  Add temporary survey to Passwords screen (#2834)
  Upload logs on cancel (#2836)
  Add bookmarks to top hits suggestions on iOS (#2835)
  Fix Maestro reporting for e2e tests (#2713)
  support marketplace kit (#2833)
  Enable attribution flag (#2825)
  Release 7.119.0-0 (#2831)
  Fix subscriptions glitch in Settings (#2828)
  Delegate SecureVault KeyStore Events in All Scenarios  (#2827)
  Clears subscription cache on restore (#2823)

# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
#	DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
samsymons added a commit that referenced this pull request May 13, 2024
* main: (27 commits)
  Release 7.120.0-0 (#2846)
  Enable gzip compression for Sync PATCH payloads (#2805)
  sync error notify (#2822)
  Wire up VPN error UI (#2826)
  Mergeback from release/7.119.0
  Release 7.119.0-1 (#2840)
  Add temporary survey to Passwords screen (#2834)
  Upload logs on cancel (#2836)
  Add bookmarks to top hits suggestions on iOS (#2835)
  Fix Maestro reporting for e2e tests (#2713)
  support marketplace kit (#2833)
  Enable attribution flag (#2825)
  Release 7.119.0-0 (#2831)
  Fix subscriptions glitch in Settings (#2828)
  Delegate SecureVault KeyStore Events in All Scenarios  (#2827)
  Clears subscription cache on restore (#2823)
  Enable field validation for Sync payloads (#2807)
  Fix up address bar spoofing tests to match new behaviours. (#2743)
  Release 7.118.0-4 (#2821)
  Settings: onAppear changed to onDisappear (#2817) (#2820)
  ...
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.

None yet

2 participants