Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

replace scrimba with khan academy #1103

Merged
merged 4 commits into from
Dec 7, 2024
Merged

Conversation

SallyMcGrath
Copy link
Member

@SallyMcGrath SallyMcGrath commented Oct 21, 2024

What does this change?

This is a proposed replacement for Scrimba.

Pros:

you can complete it on a library computer
it is 100% free with no upsells
it has videos as well as interactive coding playgrounds
it covers semantic html, devtools, deployment
it can be done in 2.5 hours and will probably take 6-10
it's not madly wrong

Cons

It's old! This is the biggest con - it's 9 years old
The projects look crap
You can't type in the video like Scrimba

Overall, it's the best replacement I've found. I looked at 15 possible courses:

w3schools, codeacademy, web.dev, mdn, html.com, educative, freecodecamp, htmldog, htmlacademy, brainstation, learnhtmlcss, learn-html, skillshop and coursera.

Basically the good ones have mega upsell problems. The bad ones are just long infodumps, or sometimes so wildly wrong it's just unhelpful. Or break off into huge lectures about stuff the learner can't possibly grok yet. Or teach emmet before teaching headings. Just - it's a struggle you guys!

Org Content?

ITD | Step 3

If we like this, maybe we pair it with build a personal home page for step 4 and get a code review for step 5.

Checklist

Who needs to know about this?

I have not replaced the next step yet - this is for people to look at and discuss.
Copy link

netlify bot commented Oct 21, 2024

Deploy Preview for cyf-programming ready!

Name Link
🔨 Latest commit ebda1e8
🔍 Latest deploy log https://app.netlify.com/sites/cyf-programming/deploys/67542423cfcfd400086b3d2a
😎 Deploy Preview https://deploy-preview-1103--cyf-programming.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 100 (no change from production)
Accessibility: 100 (no change from production)
Best Practices: 92 (no change from production)
SEO: 90 (no change from production)
PWA: -
View the detailed breakdown and full score reports

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

Copy link

netlify bot commented Oct 21, 2024

Deploy Preview for cyf-curriculum canceled.

Name Link
🔨 Latest commit ebda1e8
🔍 Latest deploy log https://app.netlify.com/sites/cyf-curriculum/deploys/67542423f7d8fe00083b6ac4

Copy link

netlify bot commented Oct 21, 2024

Deploy Preview for cyf-common ready!

Name Link
🔨 Latest commit ebda1e8
🔍 Latest deploy log https://app.netlify.com/sites/cyf-common/deploys/67542423b628880008e8a680
😎 Deploy Preview https://deploy-preview-1103--cyf-common.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

Copy link

netlify bot commented Oct 21, 2024

Deploy Preview for cyf-launch ready!

Name Link
🔨 Latest commit ebda1e8
🔍 Latest deploy log https://app.netlify.com/sites/cyf-launch/deploys/6754242353da5b0008ae7bc0
😎 Deploy Preview https://deploy-preview-1103--cyf-launch.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

Copy link

netlify bot commented Oct 21, 2024

Deploy Preview for cyf-sdc ready!

Name Link
🔨 Latest commit ebda1e8
🔍 Latest deploy log https://app.netlify.com/sites/cyf-sdc/deploys/67542423a1e2430008f1c743
😎 Deploy Preview https://deploy-preview-1103--cyf-sdc.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@SallyMcGrath SallyMcGrath requested review from cjyuan and Ara225 October 21, 2024 07:59
Copy link

netlify bot commented Oct 21, 2024

Deploy Preview for cyf-itd ready!

Name Link
🔨 Latest commit ebda1e8
🔍 Latest deploy log https://app.netlify.com/sites/cyf-itd/deploys/67542423757645000844b3da
😎 Deploy Preview https://deploy-preview-1103--cyf-itd.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 85 (🔴 down 6 from production)
Accessibility: 100 (no change from production)
Best Practices: 92 (no change from production)
SEO: 100 (no change from production)
PWA: -
View the detailed breakdown and full score reports

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

Copy link

netlify bot commented Oct 21, 2024

Deploy Preview for cyf-piscine ready!

Name Link
🔨 Latest commit ebda1e8
🔍 Latest deploy log https://app.netlify.com/sites/cyf-piscine/deploys/675424233422f40008d9bf07
😎 Deploy Preview https://deploy-preview-1103--cyf-piscine.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 85 (🔴 down 15 from production)
Accessibility: 100 (no change from production)
Best Practices: 92 (no change from production)
SEO: 100 (no change from production)
PWA: -
View the detailed breakdown and full score reports

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

Copy link

netlify bot commented Oct 21, 2024

Deploy Preview for cyf-tracks ready!

Name Link
🔨 Latest commit ebda1e8
🔍 Latest deploy log https://app.netlify.com/sites/cyf-tracks/deploys/6754242319e81600081cd19c
😎 Deploy Preview https://deploy-preview-1103--cyf-tracks.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@cjyuan
Copy link
Contributor

cjyuan commented Oct 24, 2024

I prefer to stick with Scrimba. At least I knew how it works and most participants could eventually finish the step. (Only uncertainty is whether if it will change the middle of an ITD).

Although the systematic way in which Khan Academy introduces HTML and CSS can help the participants learn better, I am reluctant to try something new 7 days before the launch of next ITD.

@Ara225
Copy link
Contributor

Ara225 commented Oct 24, 2024

I think this pull request is the right thing to do. We probably won't find perfect - the motives to charge in a company are too high and the non profits rarely have the resources to excel, so good enough is fine by me.

I agree that this is pretty close to the start and therefore major changes are bad but honestly, if we don't rip the plaster off at some point we'll keep making work arounds etc for Scrimba and it will get more and more embedded until none can contemplate a change. What if they decide to charge even for the bits we need or to change the course substantially? At least with Khan Academy they're likely to stay free because of their goals being more or less aligned with ours.

@SallyMcGrath
Copy link
Member Author

What I will do is let you two fight it out. 😉

I'll merge a version of this step as a backup, with the build turned off. And add also a personal home page step that's the same. Then you have all your options. You have the scrimba and blocks version, and if you decide to swap out now or next time, you can do that v quickly if the steps are all there ready

@SallyMcGrath SallyMcGrath marked this pull request as draft October 26, 2024 10:02
@SallyMcGrath
Copy link
Member Author

oh I don't even need to merge it really -- I can just store it here in this closed PR linked to #956

@SallyMcGrath SallyMcGrath reopened this Dec 6, 2024
@SallyMcGrath SallyMcGrath marked this pull request as ready for review December 6, 2024 18:58
@SallyMcGrath
Copy link
Member Author

@cjyuan if you can approve this I will open the next pr off this version

@cjyuan
Copy link
Contributor

cjyuan commented Dec 7, 2024

@SallyMcGrath, Replace Scrimba with Khan Academy? Yes. And how do I approve?

@SallyMcGrath
Copy link
Member Author

@SallyMcGrath, Replace Scrimba with Khan Academy? Yes. And how do I approve?

Do you mean you don't have access/perms to approve or you don't know the process to do so?

@cjyuan
Copy link
Contributor

cjyuan commented Dec 7, 2024

I was unfamiliar with the process. But Daniel Atherton helped me figure out how to do it.

Copy link
Contributor

@cjyuan cjyuan left a comment

Choose a reason for hiding this comment

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

Yes. Replace Scrimba by Khan Academy.

@SallyMcGrath SallyMcGrath merged commit 4d9057a into main Dec 7, 2024
34 checks passed
@SallyMcGrath SallyMcGrath deleted the content/replace-scrimba branch December 7, 2024 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants