Fix global links in KB Web cloud #8072
Conversation
| @@ -188,7 +188,7 @@ After 24 hours, if your `DKIM` box is red, please refer to the section [“Why d | |||
|
|
|||
| #### Full DKIM configuration <a name="firststep"></a> | |||
|
|
|||
| To configure DKIM, go to the website <https://ca.api.ovh.com/console/>, log in using the `Login`{.action} button in the top right-hand corner, and enter your OVHcloud credentials. | |||
| To configure DKIM, go to the website </links/console>, log in using the `Login`{.action} button in the top right-hand corner, and enter your OVHcloud credentials. | |||
There was a problem hiding this comment.
These <> links won't work with our /links pattern. I'll work on adapting these.
There was a problem hiding this comment.
this was my second error so ... sorry
There was a problem hiding this comment.
if you can and prefer i can remake these without <> links
There was a problem hiding this comment.
That would be much appreciated!
There was a problem hiding this comment.
so I close the pr named Fix global and I redo them avoiding the <> links ? is that good for you ? does it at least help you ?
There was a problem hiding this comment.
i can do this... just change my regex...
There was a problem hiding this comment.
so I close the pr named
Fix globaland I redo them avoiding the <> links ? is that good for you ? does it at least help you ?
I've already merged a few of them and currently checking the Hosted Private Cloud one. If you know you've got other <> links edited in some other pull requests, you can edit those PRs.
There was a problem hiding this comment.
close before rewrite without <> links
There was a problem hiding this comment.
I've closed them all because I'm an idiot working mainly by script, as you've gathered.
There was a problem hiding this comment.
well all ... no ... those that were open
DoStupidThingsFasterWithMoreEnergy
deleted the
Fix-global-links-in-kb-Web-cloud
branch
|
@Y0Coss |
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.