Skip to content
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

Remove false items #6044

Open
wants to merge 18 commits into
base: master
Choose a base branch
from
Open

Remove false items #6044

wants to merge 18 commits into from

Conversation

sharadregoti
Copy link
Contributor

@sharadregoti sharadregoti commented Mar 3, 2025

User description

For internal users - Please add a Jira DX PR ticket to the subject!



Preview Link


Description


Screenshots (if appropriate)


Checklist

  • I have added a preview link to the PR description.
  • I have reviewed the suggestions made by our AI (PR Agent) and updated them accordingly (spelling errors, rephrasing, etc.)
  • I have reviewed the guidelines for contributing to this repository.
  • I have read the technical guidelines for contributing to this repository.
  • Make sure you have started your change off our latest master.
  • For Tyk Members - Please add a Jira DX PR ticket to the subject!
  • For Tyk Members - I have added the appropriate release label to this PR:
    • If it is for a future release, label it as future-release and specify the version (e.g., future-release, 6.0).
    • If it should be merged into an older version, use the specific version label (e.g., 4.1, 5.1).
    • If no label is added, it will be assumed that the PR should be merged into the latest current version (e.g., 5.5) and master.

PR Type

  • Enhancement

Description

  • Removed deprecated and false menu items.

  • Cleaned up unused navigation entries.

  • Added new installation alias to open-source documentation.


Changes walkthrough 📝

Relevant files
Enhancement
tyk-open-source.md
Add new installation alias.                                                           

tyk-docs/content/tyk-open-source.md

  • Added alias /getting-started/installation to aliases.
+1/-0     
menu.yaml
Clean up false menu items.                                                             

tyk-docs/data/menu.yaml

  • Removed "Deployment and Operations" block with installation guide.
  • Removed "Tyk Sync" entry.
  • Removed "SDK" and "CLI" entries.
  • Removed orphan pages section.
  • +1/-35   

    Need help?
  • Type /help how to ... in the comments thread for any questions about PR-Agent usage.
  • Check out the documentation for more information.
  • Copy link
    Contributor

    github-actions bot commented Mar 3, 2025

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ Recommended focus areas for review

    Link Consistency

    The newly added alias "/getting-started/installation" should be checked to ensure it aligns with the current navigation structure, especially since a related menu entry was removed.

    - /getting-started/installation
    Formatting

    Verify that the indentation and placement of the newly added "show: True" property maintains the consistent YAML structure and does not introduce parsing issues.

    show: True

    Copy link
    Contributor

    github-actions bot commented Mar 3, 2025

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    Copy link

    netlify bot commented Mar 3, 2025

    PS. Pls add /docs/nightly to the end of url

    Name Link
    🔨 Latest commit 6f522da
    🔍 Latest deploy log https://app.netlify.com/sites/tyk-docs/deploys/67f8add0583fa70008665896
    😎 Deploy Preview https://deploy-preview-6044--tyk-docs.netlify.app
    📱 Preview on mobile
    Toggle QR Code...

    QR Code

    Use your smartphone camera to open QR code link.

    To edit notification comments on pull requests, go to your Netlify site configuration.

    @sharadregoti sharadregoti marked this pull request as draft March 7, 2025 08:59
    @sharadregoti sharadregoti marked this pull request as ready for review April 8, 2025 05:04
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Projects
    None yet
    Development

    Successfully merging this pull request may close these issues.

    2 participants