Fix global links in KB Hosted private cloud #8066
Closed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Of course, I can correct this kind of error, but that's not the purpose of this PR. |
|
close before rewrite without |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The purpose of these pull requests (Fix global links in KB ...) is to replace all links referenced in /links with their
/linksvalues.Example: replace all
https://www.ovh.com/auth/?action=gotomanager&from=https://www.ovh.co.uk/&ovhSubsidiary=GBlinks with/links/manager.I've seen that a lot have been replaced by your work, but I've also found a lot, which is why I'm making pr by kb.
These are not a priority, however, I think they can help you make the documentation source more consistent and can prevent possible errors.
Sorry for closing my last pull requests, but I noticed a first error followed by a second one... so I fixed them and for me it's now OK.
FYI it is also possible to automate it using the methods I mentioned in the issue: #8038
Be free to change the titles or to ask me any question.
Thank you for your reviews.
@Y0Coss no need for the
Fixlabel, maybeMinor Updateand even then, I'll let you be the judge.