Skip to content

web-js-tutorial.js.org #9973

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

Closed
wants to merge 1 commit into from
Closed

web-js-tutorial.js.org #9973

wants to merge 1 commit into from

Conversation

skyler1325
Copy link

@skyler1325 skyler1325 commented Jul 17, 2025

The site content is about making basic javascript for websites (eg: buttons, event handlers, and alert popups)

@MattIPv4 MattIPv4 changed the title Update cnames_active.js web-js-tutorial.js.org Jul 17, 2025
@@ -3431,6 +3431,7 @@ var cnames_active = {
"wdd": "wangduanduan.github.io",
"weather": "phanendraguptha.github.io/weather",
"weaver": "maxkfranz.github.io/weaver", // noCF? (don´t add this in a new PR)
"web-js-tutorial": "skyler1325.github.io/web-js-tutorial"
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
"web-js-tutorial": "skyler1325.github.io/web-js-tutorial"
"web-js-tutorial": "skyler1325.github.io/web-js-tutorial",

As the CI error notes, you're missing a comma at the end of your entry.

@MattIPv4
Copy link
Member

@indus I defer to you on an assessment of the content here, my gut says there is pretty limited content (just some basic alert() examples) for what seems to be a very generic subdomain.

@indus
Copy link
Member

indus commented Jul 18, 2025

@indus I defer to you on an assessment of the content here, my gut says there is pretty limited content (just some basic alert() examples) for what seems to be a very generic subdomain.

@skyler1325 A single HTML file with 7 lines of the most basic JavaScript (+ some inline expressions) - that is not what we see as "reasonable content".

@indus indus closed this Jul 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants