Skip to content

Conversation

sharadregoti
Copy link
Contributor

@sharadregoti sharadregoti commented Oct 9, 2025

User description

Contributor checklist

  • Reviewed PR Code suggestions and updated accordingly
  • Tyklings: Labled the PR with the relevant releases
  • Tyklings: Added Jira DX PR ticket to the subject

New Contributors



PR Type

Documentation, Enhancement


Description

  • Update docs baseURL to nightly path

  • Remove legacy Netlify redirects block


Diagram Walkthrough

flowchart LR
  cfg["config.toml baseURL"] -- "set to /docs/nightly/" --> nightly["Nightly docs host"]
  redirects["Netlify _redirects entries"] -- "remove legacy release mappings" --> cleanup["Simplified routing"]
Loading

File Walkthrough

Relevant files
Configuration changes
_redirects
Remove legacy version redirects                                                   

_redirects

  • Remove multiple version-specific redirect rules.
  • Drop nightly and specific release redirects for 3.x–5.10.
+0/-21   
config.toml
Point baseURL to nightly docs                                                       

tyk-docs/config.toml

  • Change baseURL from //tyk.io/docs/ to //tyk.io/docs/nightly/.
+1/-1     

Copy link
Contributor

github-actions bot commented Oct 9, 2025

⚠️ Deploy preview for PR #7047 did not become live after 3 attempts.
Please check Netlify or try manually: Preview URL

Copy link
Contributor

github-actions bot commented Oct 9, 2025

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

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

Broken Links

Changing baseURL to /docs/nightly/ can break absolute/relative links, canonical URLs, RSS, and sitemap entries. Verify that generated URLs, redirects, and assets resolve correctly in the nightly environment.

baseURL = "//tyk.io/docs/nightly/"
SEO Impact

Ensure canonical URL configuration and robots settings are aligned with the nightly docs to avoid indexing issues or duplicate content pointing to nightly.

baseURL = "//tyk.io/docs/nightly/"

Copy link
Contributor

github-actions bot commented Oct 9, 2025

PR Code Suggestions ✨

Explore these optional code suggestions:

CategorySuggestion                                                                                                                                    Impact
Possible issue
Use explicit HTTPS scheme

Using protocol-relative URLs can cause mixed-content or resolution issues in some
environments. Set an explicit HTTPS scheme to ensure consistent behavior across
builds and redirects.

tyk-docs/config.toml [1]

-baseURL = "//tyk.io/docs/nightly/"
+baseURL = "https://tyk.io/docs/nightly/"
Suggestion importance[1-10]: 7

__

Why: The suggestion accurately targets the new baseURL line and proposes a correct, low-risk improvement to avoid protocol-relative URL issues by using explicit HTTPS. It enhances reliability but is not a critical bug fix.

Medium

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.

1 participant