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

Add docs redirects #13054

Closed
wants to merge 10,000 commits into from
Closed

Add docs redirects #13054

wants to merge 10,000 commits into from

Conversation

anthonydeguzman
Copy link
Collaborator

Proposed changes:
Added redirects based off SEO recommendations

Status (please check what you already did):

  • added some tests for the functionality
  • updated the documentation
  • updated the changelog (please check changelog for instructions)
  • reformat files using black (please check Readme for instructions)

Urkem and others added 30 commits July 20, 2023 16:04
…pi-sample-in-docs

Correct action server api sample in docs
* implement changes

* fix order of args

* update tests

* add changelog entry

* update docstrings and comments, unify if checks to remove indentation

* amend ID to accept null value instead of empty string, edit docs

* fix code quality fail for changelog name

* update schema, improve debug logging
Modify docker release steps
Added space to include proper text in auto-generated docs
* use same session across requests

* add changelog file

* formatting changes and changelog name change

* fix tests and linting
---
updated-dependencies:
- dependency-name: networkx
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Bumps [marocchino/sticky-pull-request-comment](https://github.com/marocchino/sticky-pull-request-comment) from 2.6.2 to 2.7.0.
- [Release notes](https://github.com/marocchino/sticky-pull-request-comment/releases)
- [Commits](marocchino/sticky-pull-request-comment@f61b6cf...f6a2580)

---
updated-dependencies:
- dependency-name: marocchino/sticky-pull-request-comment
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Bumps [word-wrap](https://github.com/jonschlinkert/word-wrap) from 1.2.3 to 1.2.4.
- [Release notes](https://github.com/jonschlinkert/word-wrap/releases)
- [Commits](jonschlinkert/word-wrap@1.2.3...1.2.4)

---
updated-dependencies:
- dependency-name: word-wrap
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Bumps [dns-packet](https://github.com/mafintosh/dns-packet) from 1.3.1 to 1.3.4.
- [Release notes](https://github.com/mafintosh/dns-packet/releases)
- [Changelog](https://github.com/mafintosh/dns-packet/blob/master/CHANGELOG.md)
- [Commits](mafintosh/dns-packet@v1.3.1...v1.3.4)

---
updated-dependencies:
- dependency-name: dns-packet
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Bumps [minimist](https://github.com/minimistjs/minimist) from 1.2.5 to 1.2.8.
- [Release notes](https://github.com/minimistjs/minimist/releases)
- [Changelog](https://github.com/minimistjs/minimist/blob/main/CHANGELOG.md)
- [Commits](minimistjs/minimist@v1.2.5...v1.2.8)

---
updated-dependencies:
- dependency-name: minimist
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
[ATO-1326] Update python, javascript and CI action dependencies
m-vdb and others added 21 commits November 28, 2023 10:06
Remove scheduled runs of the CI GPU workflows
Delete auto main to branch merge workflow
Changes the link to Studio docs from incorrect https://rasa.com/docs/rasa-studio/ to correct https://rasa.com/docs/studio/
Finish docs redirects

Add redirects

Add redirect
Copy link
Collaborator

@m-vdb m-vdb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the main branch on RasaHQ/rasa doesn't receive updates anymore; can you point to the documentation branch instead? Your changes will be automatically deployed once the documentation branch receives new commits.

Comment on lines 206 to 208
[[redirects]]
from = "/docs/rasa-x/changelog/rasa-x-changelog/"
to = "/docs/rasa-enterprise/changelog/rasa-enterprise-changelog/"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not sure if this rule will work, it conflicts with this one on rasa-website

Comment on lines 210 to 212
[[redirects]]
from = "/docs/rasa-enterprise/changelog/rasa-x-changelog/"
to = "/docs/rasa-enterprise/changelog/rasa-enterprise-changelog/"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you move all the rasa-enterprise redirects in the rasa-x repo? here

Comment on lines 274 to 276
[[redirects]]
from = "/docs/getting-started/demobot/"
to = "/docs/"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this rule needs to be added to rasa-website for it to work

to = "/docs/rasa/action-server/sdk-tracker/"

[[redirects]]
from = "/docs/get_started_step1/"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this rule needs to be added to rasa-website for it to work

@anthonydeguzman anthonydeguzman changed the base branch from main to documentation October 28, 2024 05:39
@anthonydeguzman
Copy link
Collaborator Author

Tried changing the base branch looks like I messed up this pull request. I'll open a different one

@anthonydeguzman anthonydeguzman deleted the docs_seo_updates branch October 28, 2024 05:42
Copy link
Contributor

🚀 A preview of the docs have been deployed at the following URL: https://13054--rasahq-docs-rasa-v2.netlify.app/docs/rasa

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.