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

fix: updated baseURL for prod and staging; fixes sitemap issues #1904

Merged
merged 1 commit into from
Aug 25, 2021

Conversation

travisamartin
Copy link
Contributor

@travisamartin travisamartin commented Aug 25, 2021

Proposed changes

  • Fixed the baseURL in the config.toml for the prod and staging doc sites. The baseURL should use the full domain and path; otherwise, the sitemaps won't be formatted correctly.
  • Closes IND-27825

Current KIC sitemap:

https://docs.nginx.com/nginx-ingress-controller/sitemap.xml

image

Compare to Controller sitemap:

https://docs.nginx.com/nginx-controller/sitemap.xml

image

@github-actions github-actions bot added the bug An issue reporting a potential bug label Aug 25, 2021
@lucacome lucacome added documentation Pull requests/issues for documentation and removed bug An issue reporting a potential bug labels Aug 25, 2021
@travisamartin travisamartin merged commit 1c8e5f1 into docs-1.12 Aug 25, 2021
@travisamartin travisamartin deleted the fix-kic-sitemap-baseurl branch August 25, 2021 22:35
ciarams87 added a commit that referenced this pull request Sep 8, 2021
fix: Updates to f5-theme

fix issues with raw html code tags in docs

Update netlify.toml

Add branch deploy config for release branch
Auto-update theme, go.mod from gitlab.com/f5/nginx/controller/poc/f5-hugo. (#1842)

Co-authored-by: Travis Martin <t.martin@f5.com>
Merge branch 'staging' into add-kic-left-nav-toc

Merge pull request #1848 from nginxinc/add-kic-left-nav-toc

fix: Add left nav TOC to KIC docs (docs-1.12)
fix: updated baseURL for prod and staging; fixes sitemap issues

Merge pull request #1904 from nginxinc/fix-kic-sitemap-baseurl

fix: updated baseURL for prod and staging; fixes sitemap issues
fix: DOCOPS-122 fix broken links in KIC

Merge pull request #1936 from nginxinc/DOCOPS-122b

fix: DOCOPS-122 fix broken links in KIC and KIC-407 remove old TOCs
fix: remove toc from globalconfiguration-resource

Merge pull request #1937 from nginxinc/DOCOPS-122b

KIC-407 remove old TOC (redundant)
fix: DOCOPS-232 broken link

Merge pull request #1938 from nginxinc/DOCOPS-232

Docops 232 - Broken link in App-protect configuration doc
Add instructions for configuring MyF5 JWT as a docker registry secret (#1898)

* Add instructions for configuring MyF5 JWT as a docker registry secret

* PR feedback

Co-authored-by: nginx-bot <68849795+nginx-bot@users.noreply.github.com>
ciarams87 added a commit that referenced this pull request Sep 8, 2021
fix: Updates to f5-theme

fix issues with raw html code tags in docs

Update netlify.toml

Add branch deploy config for release branch
Auto-update theme, go.mod from gitlab.com/f5/nginx/controller/poc/f5-hugo. (#1842)

Co-authored-by: Travis Martin <t.martin@f5.com>
Merge branch 'staging' into add-kic-left-nav-toc

Merge pull request #1848 from nginxinc/add-kic-left-nav-toc

fix: Add left nav TOC to KIC docs (docs-1.12)
fix: updated baseURL for prod and staging; fixes sitemap issues

Merge pull request #1904 from nginxinc/fix-kic-sitemap-baseurl

fix: updated baseURL for prod and staging; fixes sitemap issues
fix: DOCOPS-122 fix broken links in KIC

Merge pull request #1936 from nginxinc/DOCOPS-122b

fix: DOCOPS-122 fix broken links in KIC and KIC-407 remove old TOCs
fix: remove toc from globalconfiguration-resource

Merge pull request #1937 from nginxinc/DOCOPS-122b

KIC-407 remove old TOC (redundant)
fix: DOCOPS-232 broken link

Merge pull request #1938 from nginxinc/DOCOPS-232

Docops 232 - Broken link in App-protect configuration doc
Add instructions for configuring MyF5 JWT as a docker registry secret (#1898)

* Add instructions for configuring MyF5 JWT as a docker registry secret

* PR feedback

Co-authored-by: nginx-bot <68849795+nginx-bot@users.noreply.github.com>
ciarams87 added a commit that referenced this pull request Sep 8, 2021
fix: Updates to f5-theme

fix issues with raw html code tags in docs

Update netlify.toml

Add branch deploy config for release branch
Auto-update theme, go.mod from gitlab.com/f5/nginx/controller/poc/f5-hugo. (#1842)

Co-authored-by: Travis Martin <t.martin@f5.com>
Merge branch 'staging' into add-kic-left-nav-toc

Merge pull request #1848 from nginxinc/add-kic-left-nav-toc

fix: Add left nav TOC to KIC docs (docs-1.12)
fix: updated baseURL for prod and staging; fixes sitemap issues

Merge pull request #1904 from nginxinc/fix-kic-sitemap-baseurl

fix: updated baseURL for prod and staging; fixes sitemap issues
fix: DOCOPS-122 fix broken links in KIC

Merge pull request #1936 from nginxinc/DOCOPS-122b

fix: DOCOPS-122 fix broken links in KIC and KIC-407 remove old TOCs
fix: remove toc from globalconfiguration-resource

Merge pull request #1937 from nginxinc/DOCOPS-122b

KIC-407 remove old TOC (redundant)
fix: DOCOPS-232 broken link

Merge pull request #1938 from nginxinc/DOCOPS-232

Docops 232 - Broken link in App-protect configuration doc
Add instructions for configuring MyF5 JWT as a docker registry secret (#1898)

* Add instructions for configuring MyF5 JWT as a docker registry secret

* PR feedback

Co-authored-by: nginx-bot <68849795+nginx-bot@users.noreply.github.com>

Co-authored-by: nginx-bot <68849795+nginx-bot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Pull requests/issues for documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants