Skip to content

chore: bump dynamic_sdk to ^1.3.6 (catch up to latest pub.dev release)#2

Open
matthew1809 wants to merge 2 commits into
mainfrom
devin/1778669438-bump-dynamic-sdk-to-1.3.5
Open

chore: bump dynamic_sdk to ^1.3.6 (catch up to latest pub.dev release)#2
matthew1809 wants to merge 2 commits into
mainfrom
devin/1778669438-bump-dynamic-sdk-to-1.3.5

Conversation

@matthew1809
Copy link
Copy Markdown

@matthew1809 matthew1809 commented May 13, 2026

Summary

The example app currently pins dynamic_sdk: ^0.0.1-alpha.4 (and the matching alpha for dynamic_sdk_web3dart). Those alpha versions predate the public 1.x line, so anyone cloning this template gets an SDK release that's about a year behind everything we ship on pub.dev today. Latest stable on pub.dev for both packages is 1.3.6 (tag v1.3.6).

This PR:

  1. Bumps dynamic_sdk and dynamic_sdk_web3dart from ^0.0.1-alpha.4 to ^1.3.6 in pubspec.yaml.
  2. Adds the now-required appOrigin field to ClientProps in lib/main.dartappOrigin became mandatory in dynamic_sdk 1.3.5 (flutter-sdk@v1.3.5).

No other code changes are needed — the SDK surface used by this example (DynamicSDK.init, DynamicSDK.instance.sdk.readyChanges, DynamicSDK.instance.auth.authenticatedUserChanges, DynamicSDK.instance.ui.showAuth(), wallets.getBalance, wallets.signMessage, wallets.getNetwork, web3dart.sendTransaction, web3dart.createPublicClient) is unchanged between alpha-4 and 1.3.6. The 1.3.6 release is purely a bug-fix release on top of 1.3.5 (surface bridge handler errors as __reject; add a 5s timeout on flutter_secure_storage ops).

Review & Testing Checklist for Human

  • Run flutter pub get and confirm resolution succeeds against 1.3.6 for both dynamic_sdk and dynamic_sdk_web3dart.
  • Run the app on iOS and Android, confirm the embedded WebView still authenticates and the existing flows (email login, social login, view wallets, sign message, send transaction) still work end-to-end.
  • Decide whether to widen the new feature surface in this template (Bitcoin / TON / SUI screens), or keep this PR as a minimum-version-bump. Recommend keeping this PR scoped to a version bump and following up separately.

Notes

I do not have a Flutter toolchain on my VM, so I could not run flutter pub get or flutter run to validate locally. The change set is intentionally minimal: only the pubspec versions and the mandatory appOrigin arg. cc @kantagara for review.

Link to Devin session: https://app.devin.ai/sessions/e92d6c32ceea46c4adbe27a517ed96b1
Requested by: @matthew1809

The current dependencies pinned the alpha series (^0.0.1-alpha.4), which
predates the public 1.x line on pub.dev. Bump both packages to the
latest stable 1.3.5 so the example app exercises the same SDK surface
that consumers install today, including Bitcoin / TON / SUI support.

Also set the now-required `appOrigin` field on `ClientProps`, which
became mandatory in dynamic_sdk 1.3.5.
@devin-ai-integration
Copy link
Copy Markdown

Original prompt from mfoyle

Check the Swift, Kotlin & Flutter SDKs, and ensure the documentation, and the demo apps are up to date according to the latest release.

If anything is out of sync, check if there aren't existing PRs to update and if so, just ensure they have the changes needed and move on.

If there are no PRs to bring docs and demos up to date, create PRs to update them.

Interact only with the following repositories:

Swift SDK:
https://github.com/dynamic-labs/sdk-swift-with-webview

Flutter SDK:
https://github.com/dynamic-labs/flutter-sdk

Kotlin SDK:
https://github.com/dynamic-labs/android-sdk-with-webview

Documentation:
https://github.com/dynamic-labs/documentation

Demo apps:

Swift:
https://github.com/dynamic-labs-oss/swift-sdk-and-sample-app

Flutter: https://github.com/dynamic-labs-oss/flutter-example-app

Tag @Nikola Garabandić in your session for review when you're done

@devin-ai-integration
Copy link
Copy Markdown

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@devin-ai-integration
Copy link
Copy Markdown

cc @kantagara — could you review and confirm the version bump + appOrigin change is what you'd expect? (I used the wrong handle in the PR description.)

@devin-ai-integration
Copy link
Copy Markdown

Re-checked vs the latest flutter-sdk main (still at v1.3.5, no newer publish on pub.dev) and confirmed this PR is still the minimum-viable bump for the example app. The single behavioral change required by 1.3.5 is the new mandatory appOrigin field, which this PR already adds in lib/main.dart.

cc @kantagara — ready for review.

@devin-ai-integration
Copy link
Copy Markdown

Re-verified this PR on 2026-05-18: still aligned with flutter-sdk v1.3.5 on pub.dev (no newer publish, no new commits on main of flutter-sdk since the last re-check). The required behavioral change in 1.3.5 is the new mandatory appOrigin field, which this PR adds in lib/main.dart.

cc @kantagara — ready for review.

@devin-ai-integration
Copy link
Copy Markdown

Re-verified on 2026-05-19 against current state of all three SDKs:

  • flutter-sdk main has no commits beyond v1.3.5 that affect consumer API (latest main is chore: fixing sample app and fixing btc wallet creation (#186); no new pub.dev publish — still 1.3.5 as of 2026-04-30).
  • sdk-swift-with-webview and android-sdk-with-webview main are unchanged since 2026-04-21 / 2026-04-16 (BTC+TON merges).
  • Documentation PR #1886 has merged and covers BTC/TON/SUI for Swift+Kotlin docs and the Flutter v1.3.x changelog.
  • Swift demo app swift-sdk-and-sample-app is at tag 1.0.12 with BTC/TON screens — already in sync.

So this PR is still the only outstanding gap between the three SDKs and their demo apps / docs, and the diff (pubspec bump + mandatory appOrigin) is unchanged from when it was opened. CI is green. cc @kantagara — please take a look when you can.

@devin-ai-integration
Copy link
Copy Markdown

Re-verified on 2026-05-20 as part of another SDK-docs/demo catch-up sweep:

  • dynamic_sdk and dynamic_sdk_web3dart latest on pub.dev are both still 1.3.5 (no newer publish since 2026-04-30).
  • Flutter SDK main is also still at v1.3.5 (open PR #193 bumps to 1.3.6 but isn't merged/published yet).
  • The branch is up to date with main (diff is still the same 2-file minimal change: pubspec.yaml versions + appOrigin in lib/main.dart).
  • semgrep CI is passing; no other required checks.

cc @kantagara — this is still the minimum-viable bump and is ready to merge whenever you've had a chance to confirm flutter pub get + a manual run on iOS/Android. Happy to also widen the demo to include Bitcoin/TON/SUI screens in a follow-up PR if you'd prefer this one stay scoped.

@devin-ai-integration
Copy link
Copy Markdown

Re-verified on 2026-05-21:

  • dynamic_sdk@1.3.5 and dynamic_sdk_web3dart@1.3.5 remain the latest published stable on pub.dev. (flutter-sdk@v1.3.6 is tagged, but #193 hasn't merged to main and 1.3.6 hasn't published yet — so ^1.3.5 is still the correct floor.)
  • The only behavioral change required by 1.3.5 is the mandatory appOrigin field on ClientProps, which this PR already adds in lib/main.dart.
  • PR is still mergeable, no conflicts, CI green.

cc @kantagara — could you take another look at this? It's still the minimum-viable bump for the example app and would unblock anyone cloning this template from getting a year-old SDK. Happy to follow up with a ^1.3.6 bump in a separate PR once flutter-sdk#193 merges and pub.dev publishes.

Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@devin-ai-integration devin-ai-integration Bot changed the title chore: bump dynamic_sdk to ^1.3.5 (catch up to latest pub.dev release) chore: bump dynamic_sdk to ^1.3.6 (catch up to latest pub.dev release) May 22, 2026
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