-
Notifications
You must be signed in to change notification settings - Fork 141
feat: publish new JS course #1907
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
Conversation
3ea45eb to
2c5b0c2
Compare
|
Preview for this PR was built for commit |
|
Preview for this PR was built for commit |
|
Preview for this PR was built for commit |
|
Preview for this PR was built for commit |
|
Preview for this PR was built for commit |
|
Preview for this PR was built for commit |
|
Preview for this PR was built for commit |
10f13d0 to
92d189a
Compare
|
Preview for this PR was built for commit |
|
I'm getting a ton of conflicts after #2056 merged before this one 😞 |
92d189a to
2840ebd
Compare
|
Preview for this PR was built for commit |
sources/academy/platform/expert_scraping_with_apify/solutions/integrating_webhooks.md
Show resolved
Hide resolved
|
Preview for this PR was built for commit |
|
Preview for this PR was built for commit |
1 similar comment
|
Preview for this PR was built for commit |
TC-MO
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🚀
nginx.conf
Outdated
| rewrite ^/platform/actors/development/testing-and-maintenance$ /platform/actors/development/deployment/automated-tests redirect; | ||
| rewrite ^/platform/actors/development/source-code$ /platform/actors/development/deployment/source-types redirect; | ||
|
|
||
| # Rename output schema to dataset schema |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@TC-MO I think this part is a leftover from the rebase as well 👀 I'll remove it.
…th the original URL in query
Co-authored-by: Michał Olender <[email protected]>
Co-authored-by: Michał Olender <[email protected]>
Co-authored-by: Michał Olender <[email protected]>
Co-authored-by: Michał Olender <[email protected]>
Co-authored-by: Michał Olender <[email protected]>
Co-authored-by: Michał Olender <[email protected]>
8b000c7 to
0164c83
Compare
|
Preview for this PR was built for commit |


The aim of this PR is to publish the new JS course as described in the PR description of #1584, and to unlist the old JS course. The old one should be still accessible for a grace period.
Replacing the old JS course with a new one, which is identical to the Python course, has been previously sanctioned by both Ondra and Michał.
The Plan
scraping_basics_javascriptroot leads to the new JS course.legacy/web-scraping-for-beginners. It's gonna be a read-only archive. Must benoindexto avoid cannibalization issues.web-scraping-for-beginners, i.e. the root of the old JS course URLs, leads to redirects which take people to corresponding pages in the new JS course. This lets us use the SEO juice from the old URLs.#old-js-courseto the URL. The new JS course pages contain a component which, if#old-js-courseis present in the URL, displays a commemorative plaque about the change and link the old JS course. This improves UX: "Hey, you have until 1.1.2026 to go through this course. After that please refer to the newly updated JS course ."The Plan is a result of a long discussion between Michał, Aleš, and me, which takes into account both the UX of existing users of the JS course and SEO.
Related Work
Note
Publishes the new JS course, archives the old one with redirects and an on-page notice, and updates links, content, and Nginx rewrites across the docs.
academy/webscraping/scraping_basics_javascript/*(new slugs, content, and index) and updates internal references to it.academy/webscraping/scraping_basics_legacy/*withnoindexand a legacy notice.src/components/LegacyJsCourseAdmonition.jsxand integrates it into new course pages to show a notice when?legacy-js-course=is present.'/academy/scraping-basics-javascript'.^/academy/web-scraping-for-beginners...to'/academy/scraping-basics-javascript'with?legacy-js-course=....Written by Cursor Bugbot for commit 2840ebd. Configure here.