paritytech / polkadot-sdk-docs

_THE_ Polkadot SDK Tutorial
19 stars 0 forks source link

Deprecate `docs.substrate.io` plan #26

Open Rashmirreddy opened 1 year ago

Rashmirreddy commented 1 year ago

URLs from substrate.io are quite useful and we suspect them to high a lot of traffic and clicks. But, most are not maintained, outdated, or wrong.

What we should do is:

  1. For all pages, include a fixed banner that indicates when this page was last edited. This should give a good idea if this page is already out of date or not.
  2. For pages that are going to have anew equivalent, add a banner that explicitly forward the user to the new page.

This is much better than a blanket redirect. A blanket redirect can be implemented in eg 1 year.

kianenigma commented 1 year ago

See: https://github.com/paritytech/polkadot-sdk/pull/1554#issuecomment-1722175933

kianenigma commented 11 months ago

This is essentially step 5 as described here.

sacha-l commented 11 months ago

Here's the plan I have in mind to tackle this:

  1. Add content specific banners to pages we know are outdated but we can point readers to the improved resource (e.g. like alluded to in the comments above, when Liam's storage migration ref docs are completed, point readers of https://docs.substrate.io/reference/how-to-guides/storage-migrations/basic-storage-migration/ and have a "The content of this page might have been deprecated by now, and the latest version is ... " type banner with relevant links.

  2. Provide a guide so that anyone working on new reference docs (e.g. FRAME or devrel teams) can easily make a PR to update the relevant pages on docs.substrate.io to include the content specific banner. For e.g. how to format of the banner + if they wanted to see the changes in their local environment.

Will keep this updated with specific pages I know have more useful material in the rust docs, open to discussion but to kick it off:

After not being involved in docs.substrate.io content in 1.5 years these are the ones that off the top of my head I know have better, up to date material in the Rust docs.

I'm not sure what the plan is for pointing people to the master tutorial though (as per this issue's original name). What am I missing @kianenigma ?

Beyond just adding banners, I think it can't hurt to feel free to thin down (of simply remove) a lot of the content on these docs.substrate.io pages as we work our way to adding these deprecated notes.


Another thought but unsure about how valuable it may be is to add a banner there with a general warning about content being deprecated to the most visited pages on docs.substrate.io. On Simpleanalytics for e.g. this seems to be the Tutorials page (after the actual docs.substrate.io landing page from which most people use to get into the tutorials).

Screenshot 2023-09-26 at 2 33 06 PM
sacha-l commented 11 months ago

Update - banner would look like this:

Screenshot 2023-10-02 at 12 20 05 PM

Because it's pretty catchy with the red color, I don't see the aforementioned con as being a blocker. Lmk if there's a dominant opinion against what I'm suggesting to do as a fix here.

See: https://github.com/substrate-developer-hub/substrate-docs/pull/2069

kianenigma commented 11 months ago

I think it looks good, and doesn't need to be sticky.

kianenigma commented 11 months ago

Another thought but unsure about how valuable it may be is to add a banner there with a general warning about content being deprecated to the most visited pages on docs.substrate.io. On Simpleanalytics for e.g. this seems to be the Tutorials page (after the actual docs.substrate.io landing page from which most people use to get into the tutorials).

I think just saying something outdated without a replacement might be a bit of a turn off, but I would be open to it.

sacha-l commented 11 months ago

Guide:

  1. (Optional) Follow the steps here to clone and run the docs.substrate.io site locally
  2. Make a PR to the page you want to warn readers about and copy paste this snip, adapting it to your needs:
<div class="warning">
  <strong>⚠️ WARNING:</strong> This page contains outdated information. Please refer to the <a href="<new-link-here>">Rust docs</a> for the most up-to-date documentation on this topic.
</div>
sacha-l commented 11 months ago

I think just saying something outdated without a replacement might be a bit of a turn off, but I would be open to it.

Let's stick to the yellow warning banner for now.

What are some other banners we need aka more updated docs that we can point docs.substrate.io readers to?

kianenigma commented 10 months ago

comment from @liamaharon: as soon as developer_hub is launched somewhere, we should also have a global banner that is forwarding people to the new version.

Would need to also have analytics at this point to monitor the traffic flow.

liamaharon commented 10 months ago

Is this issue about deprecating substrate.io or docs.substrate.io?

kianenigma commented 10 months ago

Is this issue about deprecating substrate.io or docs.substrate.io?

mainly docs.substrate.io, but I am 100% sure that the parent substrate.io should also be deprecated. In other words, the decision (big D) of removing substrate.io is already made. Although, the replacement for it will no longer be polkadot_sdk::developer_hub, but rather a new version of polkadot.network. Suggestions atm are dev.polkadot.network or polkadot.network/dev or polkadot.network/sdk.

Although, a page like this from the developer hub could be a short term replacement for this, but it is very crude.