-
Notifications
You must be signed in to change notification settings - Fork 135
feat: tidy up the tutorials navigation #1873
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
|
You have run out of free Bugbot PR reviews for this billing cycle. This will reset on September 24. To receive Bugbot reviews on all of your PRs, please have a team admin upgrade your team to Bugbot Pro by visiting the Cursor dashboard. Your first 14 days will be free to try! |
|
Preview for this PR was built for commit |
|
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, I appreciate the cleanup thank you!
|
Should I wait for @metalwarrior665's eyes and a seal of approval, or just ping him for awareness? |
|
I think it's not super urgent or anything, it can wait :) |
metalwarrior665
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.
Thanks, definitely an improvement. I think the Apify Scrapers might be good candidates to remove eventually, the articles are not complete (missing Playwright and Camoufox) and the Actors have really good readme. We could just change it to a single page that will list an updated list of generic scrapers. cc @nicklamonov
|
Thanks! The thing about relying more on the READMEs of the scrapers makes sense! I'll keep this in mind. Or better, I'll file an issue as I might forget about this in a month or two. |
sidebar_position:from their meta.I thought I could have a single section for Apify stuff in the tutorials, where I'd move the https://docs.apify.com/academy/running-a-web-server page, but the "Tutorials on ready-made Apify scrapers" section is too self-contained and the intro expects certain contents and ordering. I don't know exactly yet what I'll do with it, but this PR feels like a good first step, groundworks for further changes.