Skip to content

Document location permissions for the Bank and Location steps (superseded by #59) - #57

Closed
botinc-agent[bot] wants to merge 1 commit into
mainfrom
botinc/run-68e52439
Closed

botinc-agent[bot] wants to merge 1 commit into
mainfrom
botinc/run-68e52439

Conversation

@botinc-agent

@botinc-agent botinc-agent Bot commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

Summary

An earlier attempt at preparing the wrapper for the Bank and Location verification steps. #59 supersedes it.

  • README: a "Location and Bank verification steps" section (the iOS NSLocationWhenInUseUsageDescription key with a Spanish translation example, removing the Android location permissions, store disclosures, Bank and results) and a Permissions intro that mentions location.
  • Example apps (bare and Expo): the iOS location usage string.
  • Tests: the example apps' location entries, language forwarding from both entry points, completed results for each server decision, and an error type the bridge does not know.

Superseded by #59

#59 covers the same ground against the current native SDK contracts. It explains that the iOS location strings matter once an app links the native SDK's optional Location module (this pull request asks every iOS app to add them), adds the optional precise-location purpose string, ships the Spanish strings in both example apps, tests error types the native SDKs actually send, and makes the iOS bridge report retryBlocked. Prefer #59 over this pull request.

🤖 Generated with Claude Code

Opened by a BotInc run. Delivery follows the workflow's review and merge policy.
@botinc-agent botinc-agent Bot changed the title DID-5162: [H10] React Native SDK: Bank and Location steps pass-through Document location permissions for the Bank and Location steps (superseded by #59) Oct 7, 2026
@arosasg

arosasg commented Oct 11, 2026

Copy link
Copy Markdown
Collaborator

Superseded by #59, which is itself covered by the merged #60 (H10, DID-5907). Closing as part of the Console v2 PR cleanup; the commits stay at refs/pull/57/head.

@arosasg arosasg closed this Oct 11, 2026
@botinc-agent
botinc-agent Bot deleted the botinc/run-68e52439 branch October 11, 2026 10:44
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.

1 participant