Skip to content

docs: misc academy docs fixes#2536

Merged
artogahr merged 6 commits into
masterfrom
docs/fix-docs-inconsistencies
Jun 1, 2026
Merged

docs: misc academy docs fixes#2536
artogahr merged 6 commits into
masterfrom
docs/fix-docs-inconsistencies

Conversation

@artogahr

Copy link
Copy Markdown
Contributor

Hi folks, I'm going through some of the academy docs, I'll try to fix any inconsistencies I find here

@apify-service-account

apify-service-account commented May 14, 2026

Copy link
Copy Markdown
Contributor

🗑️ Preview for this PR was deleted.

@artogahr artogahr changed the title docs: fix academy ikea highest price example prices docs: misc academy docs fixes May 16, 2026
@artogahr artogahr force-pushed the docs/fix-docs-inconsistencies branch from f5db17d to d81e7d6 Compare May 19, 2026 15:47
@honzajavorek

Copy link
Copy Markdown
Collaborator

Thanks for working on this! omg, the Christmas tree 🤦‍♂️ I remember I browsed IKEA products for a while to find a category which won't have any four digit prices, so that the exercise is simple. And they add a Christmas tree to artificial plants 🔥 👿 🔥

@artogahr

Copy link
Copy Markdown
Contributor Author

the amazon frontend seems to have changed it seems, it's not possible to get more than one price for the products with crawlee since they are dynamically loaded. I don't think it's worth refactoring over, I've just added a note to https://docs.apify.com/academy/expert-scraping-with-apify/actors-webhooks

@artogahr artogahr marked this pull request as ready for review May 25, 2026 13:59
@artogahr

Copy link
Copy Markdown
Contributor Author

I'm done with the onboarding, I fixed most things I was able to find, no perfect solutions for anything but still in the spirit of "if it's better than current, ship it" 😄

@szaganek szaganek left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for the detailed update! I left some comments.

A note for the future, please make sure to check our contributing guidelines for tips on styling (like admonitions). You can also run docs-review Claude skill locally to catch some problems before opening a PR.

Comment thread sources/academy/platform/expert_scraping_with_apify/actors_webhooks.md Outdated
Comment thread sources/academy/platform/expert_scraping_with_apify/actors_webhooks.md Outdated
Comment thread sources/academy/platform/expert_scraping_with_apify/actors_webhooks.md Outdated
Comment thread sources/academy/platform/expert_scraping_with_apify/managing_source_code.md Outdated
Comment thread sources/academy/platform/expert_scraping_with_apify/managing_source_code.md Outdated
Comment thread sources/academy/webscraping/scraping_basics_python/03_devtools_extracting_data.md Outdated
Comment thread sources/academy/webscraping/scraping_basics_python/03_devtools_extracting_data.md Outdated
Comment thread sources/academy/webscraping/scraping_basics_python/03_devtools_extracting_data.md Outdated
Comment thread sources/academy/webscraping/scraping_basics_python/03_devtools_extracting_data.md Outdated
Comment thread sources/platform/integrations/data-storage/drive.md Outdated
@artogahr

Copy link
Copy Markdown
Contributor Author

Wow, thanks for the detailed review! Happy to see such strict attitude around the docs, as it should be ;)

and you're right I should've read through contributing.md more carefully before the commits :)

will go through the comments in a bit.

@artogahr

Copy link
Copy Markdown
Contributor Author

forgot the docs-review thing :|

@artogahr artogahr force-pushed the docs/fix-docs-inconsistencies branch from 366f748 to 8cde792 Compare May 29, 2026 11:52
@artogahr

artogahr commented May 29, 2026

Copy link
Copy Markdown
Contributor Author

There were a lot of small change commits so I merged them together and pushed with --force-with-lease, hope that's fine.

How do things look now?

edit: the preview build failed once despite the checks passing for some reason, I pushed an empty commit, preview built succesfully, then removed the empty commit to keep the history clean.

@artogahr artogahr force-pushed the docs/fix-docs-inconsistencies branch from bff32fe to d527f83 Compare May 29, 2026 12:17

@szaganek szaganek left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks, @artogahr! If you could just fix these three small things, we're good to go.

Comment thread sources/academy/platform/expert_scraping_with_apify/actors_webhooks.md Outdated
Comment thread sources/academy/platform/expert_scraping_with_apify/actors_webhooks.md Outdated
Comment thread sources/academy/platform/expert_scraping_with_apify/actors_webhooks.md Outdated
@artogahr artogahr merged commit de98598 into master Jun 1, 2026
16 checks passed
@artogahr artogahr deleted the docs/fix-docs-inconsistencies branch June 1, 2026 12:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants