-
Notifications
You must be signed in to change notification settings - Fork 169
docs: Update CLI menu links to point to docs page #2201
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
|
Preview for this PR was built for commit |
## Summary Remove the landing page and redirect users directly to the documentation. Users will now go to `/api/client/js/docs` instead of the landing page at `/api/client/js/`. ## Changes - Replaced landing page with redirect to `/api/client/js/docs` - Removed `website/src/pages/index.module.css` (associated styles) - Updated footer logo href to point to `docs` ## Rationale The landing page duplicated information already available in the main documentation. By removing it and redirecting to `/api/client/js/docs`, we provide a more streamlined experience for users. ## Related This PR is coordinated with: - apify/apify-docs#2201 that updates the menu links - apify/apify-cli#1003 (similar change for CLI) Co-Authored-By: Claude Sonnet 4.5 (1M context) <[email protected]>
|
Preview for this PR was built for commit |
## Summary Remove the landing page and redirect users directly to the documentation. Users will now go to `/api/client/python/docs/introduction/overview` instead of the landing page at `/api/client/python/`. ## Changes - Replaced landing page with redirect to `/api/client/python/docs/introduction/overview` - Removed `website/src/pages/index.module.css` (associated styles) - Updated footer logo href to point to `docs/introduction/overview` ## Rationale The landing page duplicated information already available in the main documentation. By removing it and redirecting to `/api/client/python/docs/introduction/overview`, we provide a more streamlined experience for users. ## Related This PR is coordinated with: - apify/apify-docs#2201 that updates the menu links - apify/apify-cli#1003 (similar change for CLI) Co-Authored-By: Claude Sonnet 4.5 (1M context) <[email protected]>
## Summary Remove the landing page and redirect users directly to the documentation. Users will now go to `/api/client/js/docs` instead of the landing page at `/api/client/js/`. ## Changes - Replaced landing page with redirect to `/api/client/js/docs` - Removed `website/src/pages/index.module.css` (associated styles) - Updated footer logo href to point to `docs` ## Rationale The landing page duplicated information already available in the main documentation. By removing it and redirecting to `/api/client/js/docs`, we provide a more streamlined experience for users. ## Related This PR is coordinated with: - apify/apify-docs#2201 that updates the menu links - apify/apify-cli#1003 (similar change for CLI) Co-Authored-By: Claude Sonnet 4.5 (1M context) <[email protected]>
## Summary Remove the landing page and redirect users directly to the documentation. Users will now go to `/api/client/python/docs/introduction/overview` instead of the landing page at `/api/client/python/`. ## Changes - Replaced landing page with redirect to `/api/client/python/docs/introduction/overview` - Removed `website/src/pages/index.module.css` (associated styles) - Updated footer logo href to point to `docs/introduction/overview` ## Rationale The landing page duplicated information already available in the main documentation. By removing it and redirecting to `/api/client/python/docs/introduction/overview`, we provide a more streamlined experience for users. ## Related This PR is coordinated with: - apify/apify-docs#2201 that updates the menu links - apify/apify-cli#1003 (similar change for CLI) Co-Authored-By: Claude Sonnet 4.5 (1M context) <[email protected]>
|
Preview for this PR was built for commit |
|
Preview for this PR was built for commit |
B4nan
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.
as said in the other PR, let's add server side redirects for all those changes too, we have nginx config in this repo. otherwise looks good (applies to all the related PRs)
|
I'll prepare a PR to add relevant redirects |
|
@B4nan Have you had change to prepare the redirects? |
|
Michal did: #2213 |
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.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
This PR is being reviewed by Cursor Bugbot
Details
Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.
To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.
Comment @cursor review or bugbot run to trigger another review on this PR
Update navigation menu to redirect CLI link from /cli/ to /cli/docs. This change applies to both the main navbar and footer navigation. Changes: - Updated navbar CLI link: /cli/ → /cli/docs - Updated footer CLI link: /cli/ → /cli/docs Co-Authored-By: Claude Sonnet 4.5 (1M context) <[email protected]>
- Updated navbar JS client link: /api/client/js/ → /api/client/js/docs - Updated footer JS client link: /api/client/js/ → /api/client/js/docs - Updated navbar Python client link: /api/client/python/ → /api/client/python/docs/introduction/overview - Updated footer Python client link: /api/client/python/ → /api/client/python/docs/introduction/overview Related to removing landing pages in apify-client-js and apify-client-python repositories. Co-Authored-By: Claude Sonnet 4.5 (1M context) <[email protected]>
Updates SDK menu links in navbar and footer to point to /docs/overview instead of landing pages, consistent with the removal of SDK landing pages. Changes: - Update JavaScript SDK link from /sdk/js/ to /sdk/js/docs/overview - Update Python SDK link from /sdk/python/ to /sdk/python/docs/overview - Remove outdated comments about trailing slashes Co-Authored-By: Claude Sonnet 4.5 (1M context) <[email protected]>
af85e92 to
fa7432c
Compare
|
Preview for this PR was built for commit |
Update SDK for Python menu links to point to the correct introduction page path. Co-Authored-By: Claude Sonnet 4.5 (1M context) <[email protected]>
|
Preview for this PR was built for commit |
Revert SDK for Python menu links back to /overview page instead of /introduction. Co-Authored-By: Claude Sonnet 4.5 (1M context) <[email protected]>
|
Preview for this PR was built for commit |
Summary
Update navigation menu to redirect CLI link from
/cli/to/cli/docs. This change applies to both the main navbar and footer navigation.Changes
/cli/→/cli/docs/cli/→/cli/docsRationale
The CLI landing page is being removed (see apify/apify-cli#1003). By updating the menu links to point directly to
/cli/docs, we ensure users are taken to the documentation instead of a non-existent landing page.Related
This PR is coordinated with apify/apify-cli#1003 that removes the landing page.
🤖 Generated with Claude Code
Note
Low Risk
Config-only link changes; risk is limited to potential broken/incorrect URLs in site navigation.
Overview
Updates Docusaurus navigation links in
apify-docs-theme/src/config.jsto point directly at documentation entry pages instead of legacy landing-page URLs.Specifically retargets the navbar and footer links for API clients (JS/Python), SDKs (JS/Python), and CLI (now
/cli/docs), removing reliance on trailing-slash redirects.Written by Cursor Bugbot for commit af85e92. Configure here.