-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Migrate to docs.substrate.io
throughout
#10007
Conversation
Why not linking flat substrate.io there? Should be the better landing page when just wanting to learn what it is. |
Yeah. This field is named homepage, so it should link to substrate.io |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - needs cargo fmt
@nukemandan I'd like to remove Google Analytics and update the script with Simple Analytics (cookie-less), but will need the exact links where it's being used. |
Moved to I ran
This right? |
Try doing a cargo fmt with 2021-09-01 nightly (which seems to be the nightly that it's using, though confusingly there seem to be two nightly versions installed). |
My sincere apologies: I did not see the A7 tag here the bot added! Does this need to be reverted? What Polkadot PR is needed that triggered this? At least this I figure: paritytech/polkadot#4076 Also, what should be noted in the next release? Should this be in the change log at all? |
I don't know why it added that flag. The polkadot companion build was fine - everything was green before merge so you should be good (esp as this is just docs change). |
homepage = "https://substrate.io"
(this is almost all of the file changes)v3
in the URL, for new versions of the docs we need to replace this manually (unless I am missing a redirect @imadarai @sacha-l ?)homepage = "https://substrate.io/"
, correct links #10006(#10017 created in this PR)