Fix dead Blast fork URL in vesu/page.tsx comment - #28
Merged
Conversation
Blast's public Starknet RPC has been decommissioned. This was the last dead-Blast reference we own (skipped earlier while PR #22 was open on this file). Swaps the example fork command in the comment for the same Alchemy endpoint already merged into README.md.
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
1 participant
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
Last dead-Blast reference we own.
vesu/page.tsx:69's comment gives the fork command a learner copies verbatim to run the vesu lesson:Blast's public Starknet RPC has been decommissioned (traced earlier: it returns a JSON-RPC error rather than a connection failure, which surfaces to a learner as
starknet-devnetcrashing withdata did not match any variant of untagged enum JsonRpcResponse— reads like devnet is broken, not like a dead endpoint). Skipped in the earlier README-wide fix pass because PR #22 was open on this exact file; #22 is now merged, so this is the last piece.Replaced only the URL, with the same Alchemy endpoint already merged into
README.md:50on this branch — verified character-for-character identical. Comment text,fromBlock: 1540452n, and all surrounding code are untouched.Proof
git rev-parse HEADat branch creation:f654e58...(matchesorigin/step-3post-Fix stale @starknet-react/core imports and cairo test arity mismatch #22-merge)packages/nextjs/app/vesu/page.tsxonly, exactly 1 line changedREADME.md's merged fork URL on this same branch — character-for-character matchgit status --short: clean apart from the intended changeTest plan
git diff --stat— 1 file, 1 line