polkadot-fellows / RFCs

Proposals for change to standards administered by the Fellowship.
https://polkadot-fellows.github.io/RFCs/
Creative Commons Zero v1.0 Universal
114 stars 55 forks source link

Introduce RFC mdBook web page #60

Closed rzadp closed 9 months ago

rzadp commented 9 months ago

Screenshot 2023-12-19 at 13 55 05


What it does

ggwpez commented 9 months ago

Is this the final URL https://paritytech.github.io/RFCs/ ?
Then we should probably mention it in the README or even link it in the GitHub repo.

rzadp commented 9 months ago

The final URL is: https://polkadot-fellows.github.io/RFCs/

The paritytech one has served as a preview, before it was configured in polkadot-fellows.

ggwpez commented 9 months ago

The final URL is: https://polkadot-fellows.github.io/RFCs/

The paritytech one has served as a preview, before it was configured in polkadot-fellows.

Thanks! @wirednkod we can probably also integrate this into the official website i hope.

wirednkod commented 9 months ago

The final URL is: polkadot-fellows.github.io/RFCs The paritytech one has served as a preview, before it was configured in polkadot-fellows.

Thanks! @wirednkod we can probably also integrate this into the official website i hope.

We already have this: https://polkadot-fellows.github.io/dashboard/#/rfcs that shows all the open RFCs. I am thinking that probably we can create either a link to "more details" from the official website, or merge the 2 and build the "cargo book" and then release it with the site. The only thing I can see as a "problem" (not really one but to the eyes of a front-end dev it could be) - is the theme and design incosistency, but that could be fixed I guess. @rzadp What do you think?

rzadp commented 8 months ago

Hey @wirednkod, I didn't know about this dashboard 👍

merge the 2 and build the "cargo book" and then release it with the site

Do you have an idea on how we could do it in details? I don't think the output of mdBook is gonna work properly inside a Vite SPA application, even forgetting about the design incosistency.

I see that the dashboard is fetching the PRs dynamically from Github. What if it fetched the RFC data from the mdBook Github branch instead? The branch is gh-pages, which gets published to Github Pages. I'm thinking I could add a json file with the data, so it would be a single fetch.

That way, the dashboard would operate on the same raw data, but present it with its own front-end design. What do you think about this approach?

Unless we want to simply go with a link and keep only one site with full details - that seems reasonable too.

wirednkod commented 8 months ago

@rzadp mdBook is nothing more than generated html pages. If an iFrame (yuk) is added inside the fellowship site, then this would work. The point here is that the data on the mdbook are really detailed and nice - and I think, its better to add a link (to a new tab) towards the RFCs page that you creaed, instead of re-inventing the wheel.

This way the mdbook would be part of the main page and still keep its own design and the work you have done.

I would like to suggest some changes to what you have if we agreed to go with the aforrementioned direction:

I can add those as issues at https://github.com/polkadot-fellows/RFCs/

rzadp commented 8 months ago

Sounds good, I agree with you.

Move Newly Proposed and Proposed categories above the Approved ones

Makes total sense, no need to create an issue, I will do it right away.

remove introduction page

It doesn't seem right for me, for a user who stumbles upon this link to not have any kind of description of what he's looking at. Are you sure about it? If the problem is that it's too short (a single sentence at the moment), we could add more to it.

wirednkod commented 8 months ago

It doesn't seem right for me, for a user who stumbles upon this link to not have any kind of description of what he's looking at. Are you sure about it? If the problem is that it's too short (a single sentence at the moment), we could add more to it.

Yeah - now that I rethink it - it does make sense for it to exist. So we are good. Thank you so much @rzadp

wirednkod commented 8 months ago

@rzadp I have integrated now the RFC-book inside the dashboard (see sceenshot): image