Skip to content

feat(network-details): [test-app] Add testing UI to iOS-Swift#7581

Open
43jay wants to merge 9 commits intomobile-935/sdk-optionsfrom
mobile-935/ios-swift-app
Open

feat(network-details): [test-app] Add testing UI to iOS-Swift#7581
43jay wants to merge 9 commits intomobile-935/sdk-optionsfrom
mobile-935/ios-swift-app

Conversation

@43jay
Copy link
Copy Markdown
Collaborator

@43jay 43jay commented Mar 3, 2026

📜 Description

Enables network detail capture for httpbin.org:

networkDetailAllowUrls = [ "httpbin.org" ]

Adds a new screen to iOS-Swift sample app to

  • Initiate different types of network requests - JSON, form-url-encoded, text, binary.
  • Show the request/response details in the test app for easy comparison against request/response details added to session replay capture.

💡 Motivation and Context

100% coded by claude, acceptance criteria was to pass swiftlint for any code i introduced.

1) Navigating to Network Details screen from Extras
image

2) Test screen on initial load
image

3) Test Screen with request/response details populated (JSON request)
image

4) Corresponding Session Replay captured (https://sentry-sdks.sentry.io/explo...)
image

💚 How did you test it?

Build + Install

📝 Checklist

You have to check all boxes before merging:

  • I added tests to verify the changes. N/A - test app
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled. N/A - test app
  • I updated the docs if needed. N/A - test app
  • I updated the wizard if needed. N/A - test app
  • Review from the native team if needed. N/A - test app
  • No breaking change or entry added to the changelog. N/A - test app #skip-changelog
  • No breaking change for hybrid SDKs or communicated to hybrid SDKs. N/A - test app

@linear
Copy link
Copy Markdown

linear bot commented Mar 3, 2026

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 3, 2026

Semver Impact of This PR

🟡 Minor (new features)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


This PR will not appear in the changelog.


🤖 This preview updates automatically when you update the PR.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 3, 2026

Messages
📖 Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against 1865d4a

Copy link
Copy Markdown
Member

@philprime philprime left a comment

Choose a reason for hiding this comment

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

Good progress, left a couple of comments

@43jay 43jay force-pushed the mobile-935/ios-swift-app branch from 0898913 to 240ba73 Compare March 4, 2026 21:03
43jay added a commit that referenced this pull request Mar 4, 2026
@43jay
Copy link
Copy Markdown
Collaborator Author

43jay commented Mar 4, 2026

@philprime PTAL 🙏

AFAICT all comments resolved except for the 'why did you not use xcode for this' ones:

@43jay 43jay requested a review from philprime March 4, 2026 22:02
@43jay 43jay force-pushed the mobile-935/ios-swift-app branch from cd8e4f0 to 327fd2e Compare March 9, 2026 19:02
@43jay 43jay force-pushed the mobile-935/sdk-options branch from cfc450a to 1d0cc82 Compare March 9, 2026 19:02
@43jay 43jay force-pushed the mobile-935/ios-swift-app branch from 327fd2e to faac0a4 Compare March 9, 2026 19:48
@43jay 43jay force-pushed the mobile-935/sdk-options branch from 1d0cc82 to 037f0e4 Compare March 9, 2026 19:48
Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

@43jay 43jay force-pushed the mobile-935/sdk-options branch from 037f0e4 to cad52c2 Compare March 10, 2026 21:06
@43jay 43jay force-pushed the mobile-935/ios-swift-app branch from 8a5da47 to 719553e Compare March 10, 2026 21:06
@43jay 43jay force-pushed the mobile-935/sdk-options branch from cad52c2 to 83310fd Compare March 16, 2026 20:31
@43jay 43jay force-pushed the mobile-935/ios-swift-app branch from 719553e to 1865d4a Compare March 16, 2026 20:31
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.

3 participants