www: refactor `docs/[...slug].tsx` to make use of optional dynamic params

mbhrznr commented 1 year ago

depends on #1706.

once merged, it would be nice to refactor www's docs/[...slug].tsx to make use of the new concept.

yossydev commented 11 months ago

@marvinhagemeister Hello! Is this issue still existing? If you don't mind, I would like to work on this problem myself!

marvinhagemeister commented 11 months ago

yes, this is still open. Feel free to go ahead.

yossydev commented 11 months ago

Thank you!! I will tackle it!!

Mrashes commented 4 months ago

@yossydev Are you still working on this? I can take a look if not

yossydev commented 4 months ago

@Mrashes Sorry, I haven't got around to it at the moment, please do!

Mrashes commented 4 months ago

Alright, started digging for an hour. Few things to start calling out:

can't get the optional routing to work out of the box (idea is docs/[[version]]/[...slug]). It results in too many redirects when I try to hit docs/introduction
Its a bit tricky due to the middleware so I'm going to try and ignore that for now
Docs are a bit out of date so I'll update those

Next step is to just get it working where when I hit docs/latest/introduction it should render locally. Probably just need more head banging but wanted to update on progress. Hope is to do this an hour at a time for the next few nights.

denoland / fresh

www: refactor `docs/[...slug].tsx` to make use of optional dynamic params #1750