Skip to content

fix: remove deprecated State Channel Application tutorial#15748

Merged
wackerow merged 1 commit into
devfrom
claude/issue-15747-20250630_133022
Jul 15, 2025
Merged

fix: remove deprecated State Channel Application tutorial#15748
wackerow merged 1 commit into
devfrom
claude/issue-15747-20250630_133022

Conversation

@konopkja
Copy link
Copy Markdown
Contributor

Remove the "State Channel Application" tutorial that links to https://speedrunethereum.com/challenge/state-channels which returns 404.

Fixes #15747

Generated with Claude Code

Remove the "State Channel Application" tutorial that links to 
https://speedrunethereum.com/challenge/state-channels which returns 404.

Fixes #15747

Co-authored-by: Jakub <konopkja@users.noreply.github.com>
@netlify
Copy link
Copy Markdown

netlify Bot commented Jun 30, 2025

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit 19e0f0d
🔍 Latest deploy log https://app.netlify.com/projects/ethereumorg/deploys/68629395b1f7be00083e1360
😎 Deploy Preview https://deploy-preview-15748--ethereumorg.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
7 paths audited
Performance: 51 (🔴 down 2 from production)
Accessibility: 95 (🟢 up 1 from production)
Best Practices: 91 (🔴 down 1 from production)
SEO: 99 (no change from production)
PWA: 59 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@konopkja
Copy link
Copy Markdown
Contributor Author

@coderabbitai can you review

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Jun 30, 2025

Tip

For best results, initiate chat on the files or code changes.

🧩 Analysis chain

@konopkja I'll review the PR for you. Let me first gather information about the changes made in this pull request.


🏁 Script executed:

#!/bin/bash
# Get PR information
gh pr view 15748 --repo ethereum/ethereum-org-website --json title,body,headRefName,baseRefName,author,files

# Show the diff for the PR
gh pr diff 15748 --repo ethereum/ethereum-org-website

Length of output: 1659


@konopkja This is a clean and well-executed maintenance fix! Here's my review:

Approved

What this PR does well:

  • Clear purpose: Removes a tutorial with a broken link (404 error)
  • Proper scope: Focused change that addresses issue Bug report: remove State Channel Application tutorial card #15747 directly
  • Clean implementation: Complete removal of the tutorial entry with proper JSON structure maintained
  • Good documentation: PR description clearly explains the rationale

Code Review:

The JSON modification in src/data/externalTutorials.json is correct:

  • ✅ Complete tutorial entry removed (all 12 lines)
  • ✅ No trailing comma issues
  • ✅ JSON structure remains valid
  • ✅ No orphaned fields or incomplete removal

Removed Tutorial Details:

  • URL: https://speedrunethereum.com/challenge/state-channels (confirmed 404)
  • Title: "State Channel Application"
  • Author: Austin Griffith
  • Skill Level: Intermediate
  • Tags: Multiple relevant tags (solidity, smart contracts, react, etc.)

This is exactly the type of maintenance work that keeps the documentation quality high. The broken link would have been frustrating for users, and removing it improves the overall user experience.

Recommendation: ✅ Ready to merge

@wackerow wackerow merged commit de474ed into dev Jul 15, 2025
7 checks passed
@wackerow wackerow deleted the claude/issue-15747-20250630_133022 branch July 15, 2025 22:59
This was referenced Jul 16, 2025
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.

Bug report: remove State Channel Application tutorial card

2 participants