[Tophat] Add Bitrise client for Tophat artifact discovery#419
Draft
Conversation
11 tasks
Contributor
Author
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
592f5f4 to
b12780f
Compare
f0b9a8f to
97464ef
Compare
b12780f to
bc818ae
Compare
97464ef to
eb98c76
Compare
bc818ae to
3f0ae1c
Compare
eb98c76 to
17aa527
Compare
17aa527 to
c32b0f5
Compare
3f0ae1c to
5aab839
Compare
This was referenced Jul 10, 2026
c32b0f5 to
9d98b2e
Compare
5aab839 to
0d2605d
Compare
Assisted-By: devx/96a590ff-685a-49ce-8199-0810f0ac2379
Assisted-By: devx/96a590ff-685a-49ce-8199-0810f0ac2379
Assisted-By: devx/96a590ff-685a-49ce-8199-0810f0ac2379
0d2605d to
96fe2c9
Compare
9d98b2e to
81d2278
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

What changes are you making?
In #401 the UX for a first-time user would be
This required multiple app hops and I wanted to unify this down a bit so now the developer experience is going to be:
In this case the token is stored into keychain and gets pushed into the keychain entry for top hat, authorizing top hat to access it. This means the user no longer has to open top hat settings in order to set up a token and streamline the process a bit. they just have to sign in on a permission prompt
How to test
Delete the bitrise token in tophat - then run
dev tophatand confirm that it launches bitrise to create a pat, and after pasting it in and pressing enter you receive a permission prompt for tophat, after that the app should installBefore you merge
Important
platforms/swift/README.mdand/orplatforms/android/README.md)Releasing a new Swift version?
ShopifyCheckoutKit.podspecplatforms/swift/Sources/ShopifyCheckoutKit/ShopifyCheckoutKit.swiftplatforms/swift/README.md(major version only)Releasing a new Embedded Checkout Protocol version?
embeddedCheckoutProtocolAndroidinplatforms/android/gradle/libs.versions.tomlprotocol/languages/kotlin/embedded-checkout-protocol/api/embedded-checkout-protocol.apiif the public API changedReleasing a new Android version?
checkoutKitAndroidinplatforms/android/gradle/libs.versions.tomlplatforms/android/README.mdTip
See the Contributing documentation for the full release process per platform.