Skip to content

feat: update the DevTools 1 lesson of the JS2 course to be about JavaScript #1654

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

honzajavorek
Copy link
Collaborator

A part of #1584

@honzajavorek honzajavorek requested a review from TC-MO June 27, 2025 13:50
@honzajavorek honzajavorek added the t-academy Issues related to Web Scraping and Apify academies. label Jun 27, 2025
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Bug: JavaScript Misclassified in Web Tech Summary

The sentence "Apart from JavaScript, websites are built with two main technologies: HTML and CSS" is factually incorrect and logically inconsistent. Websites are built with three main technologies: HTML, CSS, and JavaScript. The current phrasing incorrectly implies JavaScript is separate from the core web technologies, contradicting established web development facts and the lesson's content.

sources/academy/webscraping/scraping_basics_javascript2/01_devtools_inspecting.md#L32-L33

Apart from JavaScript, websites are built with two main technologies: HTML and CSS. In the **Elements** tab, DevTools shows the HTML and CSS of the current page:

Fix in Cursor


Comment bugbot run to trigger another review on this PR
Was this report helpful? Give feedback by reacting with 👍 or 👎

@apify-service-account
Copy link

Preview for this PR was built for commit 27aecab and is ready at https://pr-1654.preview.docs.apify.com!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
t-academy Issues related to Web Scraping and Apify academies.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants