Skip to content

Conversation

honzajavorek
Copy link
Collaborator

Previously

  • Python course had an images folder with images
  • New JS course (unlisted) had an images folder as well, but it was just a symlink to the Python one
  • Each of the courses had it's own _exercises.mdx

After this change

  • There will be three top level folders: scraping_basics with shared stuff, scraping_basics_javascript2 for the JS course, scraping_basics_python for the Python course
  • The scraping_basics contains shared images and partials (currently just _exercises.mdx, but there is a potential to share more in the future if we want)
  • The JS and Python courses link to the shared folder for images and partials
  • Only one shared _exercises.mdx, no duplication
  • No symlinks or other hacks

Note

This change doesn't touch the original scraping_basics_javascript course at all.

@honzajavorek honzajavorek requested a review from TC-MO September 4, 2025 13:16
@honzajavorek honzajavorek added the t-academy Issues related to Web Scraping and Apify academies. label Sep 4, 2025
@apify-service-account
Copy link

Preview for this PR was built for commit 8f903c5 and is ready at https://pr-1889.preview.docs.apify.com!

@honzajavorek honzajavorek mentioned this pull request Sep 8, 2025
5 tasks
@honzajavorek honzajavorek force-pushed the honzajavorek/restructure branch from 8f903c5 to 7d2a9ba Compare October 14, 2025 09:30
@apify-service-account
Copy link

Preview for this PR was built for commit 7d2a9ba and is ready at https://pr-1889.preview.docs.apify.com!

@honzajavorek
Copy link
Collaborator Author

I updated the changes so they're against current master. I fixed a few Vale errors.

@apify-service-account
Copy link

Preview for this PR was built for commit 17c47b4 and is ready at https://pr-1889.preview.docs.apify.com!

Copy link
Contributor

@TC-MO TC-MO left a comment

Choose a reason for hiding this comment

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

LGTM, hopefully Vale won't break... again

@honzajavorek honzajavorek merged commit 930242a into master Oct 15, 2025
9 checks passed
@honzajavorek honzajavorek deleted the honzajavorek/restructure branch October 15, 2025 12:26
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.

3 participants