Fix web interface React 19 dependency install#8
Open
SwiftAdviser wants to merge 2 commits into
Open
Conversation
Author
|
Update for latest commit The original PR fixed the React 19 dependency/install blocker. After This commit fixes that SDK drift:
Fresh validation:
Scope note: this makes the sample install/build clean. It does not claim the full Telegram purchase journey is E2E complete; that still needs disposable Telegram/webhook/payment/fulfillment test fixtures. |
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.
Summary
web-interface/package-lock.jsonsonpm ciis reproducibleWhat changed in the latest commit (
29fe71f)useAuthimports with the currentuseCrossmintAuthhookCrossmintWalletProvider.createOnLoginfrom the oldsignerfield to the currentrecoveryfieldWhy this matters
Before this PR, a developer following the Telegram bot quickstart could not reach the app runtime:
npm cifailed becausepackage.jsonandpackage-lock.jsonwere out of sync.npm run buildexposed SDK drift: removeduseAuth, changed wallet creation config, and changed signer methods/types.This PR moves the sample from install/build failure to a working local build baseline.
Validation
cd web-interface && npm cicd web-interface && npm run buildwith staging-safe local env valuesBuild result: PASS. Next.js still emits non-fatal transitive wallet-library warnings for optional
@react-native-async-storage/async-storageandpino-pretty, but compilation, type-checking, and route generation complete successfully.Scope / non-goals