ci(maestro): bump WalletConnect/actions for redesigned /collect IC form by ignaciosantise · Pull Request #556 · reown-com/react-native-examples

ignaciosantise · 2026-06-29T21:06:11Z

Bump `WalletConnect/actions` maestro refs for the redesigned `/collect` IC form

WalletConnect/buyer-experience PR #914 redesigns the hosted KYC webview (the /collect route the sample wallet opens in a WebView). The native Maestro pay flows drive that webview by visible text, so they were updated in WalletConnect/actions#106 (submit is now "Confirm", the "Confirm your details" dialog is gone, and consent is an inline "I agree" checkbox). The verified KYC sequence is: scroll to "I agree" → tap it → scroll to "Confirm" → tap it.

This PR pins every WalletConnect/actions/maestro/* ref (pay-tests / setup / permit2-reset) to the merged-master SHA that contains those flow changes.

Pinned SHA: ddc7c0f9acd8b1576e9a78f0ef67f4b8fbb36db2 (squash-merge of fix(maestro/pay-tests): adapt KYC flows to redesigned /collect inline IC form WalletConnect/actions#106, now master tip).

✅ Ready to merge — remaining requirement

fix(maestro/pay-tests): adapt KYC flows to redesigned /collect inline IC form WalletConnect/actions#106 is merged to master. ✔
buyer-experience #914 must be deployed to the env this wallet's /collect webview loads (prod/staging) before the pay-tagged Maestro runs will pass — the flows now expect the redesigned form.

Validation

Android Maestro pay runs were dispatched on this branch against the merged action and exercised the new KYC flow (scroll + tap "I agree", scroll + tap "Confirm").

🤖 Generated with Claude Code

…llect IC form Provisionally pinned to the branch tip of WalletConnect/actions#106 (KYC /collect inline IC form redesign). Re-pin to the merged-master SHA before merging, and only after buyer-experience #914 is deployed. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

vercel · 2026-06-29T21:06:17Z

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
deposit-app-demo	Building	Preview, Comment	Jul 3, 2026 8:52pm
pos-demo	Ready	Preview, Comment	Jul 3, 2026 8:52pm

claude · 2026-06-29T21:06:35Z

Claude finished @ignaciosantise's task in 1m 9s —— View job

Review

✅ No issues found

The diff bumps three pinned SHA refs (pay-tests, setup, permit2-reset) inside the composite action to a6436dfbb6404e7c086b24054821eb8efcc41b22. No inputs, outputs, or step IDs changed — consumer interface is intact.

The provisional-SHA situation (branch tip vs. merged-master) is correctly called out in inline comments on lines 433, 436, and 605 and is the central sequencing concern in the PR description. No additional flags from the breaking-changes or data-classification checks.

Still provisional — re-pin to the merged-master SHA before merge. Advances a6436df -> bb4ed70 so CI exercises the text-based consent ("0") and "Confirm" selectors; the earlier testIDs were not exposed in the /collect WebView. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

…ct-ic-redesign # Conflicts: # .github/actions/walletkit-build-and-maestro/action.yml

…merged) #106 now has master (#107 skip-metrics) merged in, so this single SHA carries both the redesigned /collect flow fixes and the skip-metrics marker support. Still provisional — re-pin to the merged-master SHA once #106 lands. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

#106 now taps the "I agree" consent label instead of the "0" value node, which did not toggle consent in CI. Still provisional — re-pin to the merged-master SHA once #106 lands. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

#106 now scrolls until the Confirm button is visible (off-screen on small devices). Still provisional — re-pin to merged-master SHA once #106 lands. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

#106 now uses the verified KYC sequence (scroll+tap 'I agree', scroll+tap 'Confirm'). Still provisional — re-pin to merged-master SHA once #106 lands. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

The redesigned /collect IC form is taller than the viewport, but the webview clipped content two ways: scrollEnabled/nestedScrollEnabled were false, and the injected viewport CSS forced `overflow: hidden` on html/body. Together they made the consent checkbox + Confirm button unreachable (and Maestro scrolls no-op'd) on Android. Enable scrollEnabled/nestedScrollEnabled and switch the injected CSS to `overflow-x: hidden; overflow-y: auto` (keeping overscroll-behavior:none to avoid rubber-banding). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

WalletConnect/actions#106 merged to master (ddc7c0f) — the redesigned /collect inline IC flow fixes. Pin to the merged-master SHA and drop the provisional note. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Copilot

Pull request overview

Updates the RN CLI wallet’s hosted /collect WebView + CI Maestro setup to align with the redesigned inline IC (/collect) flow, by enabling scrolling in the in-app WebView and pinning the shared WalletConnect/actions Maestro flows to the SHA that contains the updated text-driven interactions.

Changes:

Enable vertical scrolling for the hosted /collect WebView (CSS + react-native-webview scrolling props).
Pin WalletConnect/actions/maestro/* composite-action dependencies (pay-tests, setup, permit2-reset) to ddc7c0f9….

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
wallets/rn_cli_wallet/src/modals/PaymentOptionsModal/CollectDataWebView.tsx	Allows the hosted `/collect` flow to scroll within the WebView (needed for the new inline consent + confirm interactions).
.github/actions/walletkit-build-and-maestro/action.yml	Pins the shared Maestro action refs to the updated SHA; adjusts the Permit2 reset step condition.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

-      if: ${{ !cancelled() && contains(inputs.maestro-tags, 'pay') && !contains(inputs.maestro-exclude-tags, 'pay-usdt-polygon') }}
-      # Pinned to the WalletConnect/actions#105 squash-merge SHA (4cec02a on master).
-      uses: WalletConnect/actions/maestro/permit2-reset@4cec02a4c535c254581471df287099a26d5eeffb
+      if: always() && contains(inputs.maestro-tags, 'pay')


vercel Bot deployed to Preview – pos-demo June 29, 2026 21:07 View deployment

vercel Bot had a problem deploying to Preview – deposit-app-demo July 3, 2026 16:03 Failure

vercel Bot deployed to Preview – pos-demo July 3, 2026 16:04 View deployment

Merge remote-tracking branch 'origin/main' into ci/bump-actions-colle…

5d47113

…ct-ic-redesign # Conflicts: # .github/actions/walletkit-build-and-maestro/action.yml

vercel Bot had a problem deploying to Preview – deposit-app-demo July 3, 2026 16:11 Failure

vercel Bot deployed to Preview – pos-demo July 3, 2026 16:13 View deployment

vercel Bot had a problem deploying to Preview – deposit-app-demo July 3, 2026 16:14 Failure

vercel Bot deployed to Preview – pos-demo July 3, 2026 16:16 View deployment

vercel Bot had a problem deploying to Preview – deposit-app-demo July 3, 2026 18:20 Failure

vercel Bot deployed to Preview – pos-demo July 3, 2026 18:22 View deployment

ci(maestro): re-pin WalletConnect/actions to e823a98 (#106 tip)

1e1c3c2

#106 now scrolls until the Confirm button is visible (off-screen on small devices). Still provisional — re-pin to merged-master SHA once #106 lands. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

vercel Bot had a problem deploying to Preview – deposit-app-demo July 3, 2026 18:57 Failure

vercel Bot deployed to Preview – pos-demo July 3, 2026 18:59 View deployment

ci(maestro): re-pin WalletConnect/actions to bf68e7d (#106 tip)

5c9e3d5

#106 now uses the verified KYC sequence (scroll+tap 'I agree', scroll+tap 'Confirm'). Still provisional — re-pin to merged-master SHA once #106 lands. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

vercel Bot had a problem deploying to Preview – deposit-app-demo July 3, 2026 19:49 Failure

vercel Bot deployed to Preview – pos-demo July 3, 2026 19:51 View deployment

vercel Bot had a problem deploying to Preview – deposit-app-demo July 3, 2026 20:24 Failure

vercel Bot deployed to Preview – pos-demo July 3, 2026 20:26 View deployment

ci(maestro): pin WalletConnect/actions to ddc7c0f (merged #106)

68efa3e

WalletConnect/actions#106 merged to master (ddc7c0f) — the redesigned /collect inline IC flow fixes. Pin to the merged-master SHA and drop the provisional note. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

vercel Bot had a problem deploying to Preview – deposit-app-demo July 3, 2026 20:51 Failure

ignaciosantise marked this pull request as ready for review July 3, 2026 20:52

Copilot AI review requested due to automatic review settings July 3, 2026 20:52

Copilot started reviewing on behalf of ignaciosantise July 3, 2026 20:52 View session

vercel Bot deployed to Preview – pos-demo July 3, 2026 20:52 View deployment

Copilot AI reviewed Jul 3, 2026

View reviewed changes

ignaciosantise merged commit 9e67d09 into main Jul 3, 2026
15 of 16 checks passed

ignaciosantise deleted the ci/bump-actions-collect-ic-redesign branch July 3, 2026 21:14

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

ci(maestro): bump WalletConnect/actions for redesigned /collect IC form#556

ci(maestro): bump WalletConnect/actions for redesigned /collect IC form#556
ignaciosantise merged 9 commits into
mainfrom
ci/bump-actions-collect-ic-redesign

ignaciosantise commented Jun 29, 2026 •

edited

Loading

Uh oh!

vercel Bot commented Jun 29, 2026 •

edited

Loading

Uh oh!

claude Bot commented Jun 29, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

ignaciosantise commented Jun 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Bump WalletConnect/actions maestro refs for the redesigned /collect IC form

✅ Ready to merge — remaining requirement

Validation

Uh oh!

vercel Bot commented Jun 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

claude Bot commented Jun 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

ignaciosantise commented Jun 29, 2026 •

edited

Loading

Bump `WalletConnect/actions` maestro refs for the redesigned `/collect` IC form

vercel Bot commented Jun 29, 2026 •

edited

Loading

claude Bot commented Jun 29, 2026 •

edited

Loading