Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding contents from Wiki of expressjs:express in en language #1565

Open
wants to merge 8 commits into
base: gh-pages
Choose a base branch
from

Conversation

IamLizu
Copy link
Member

@IamLizu IamLizu commented Aug 10, 2024

Overview

As we have discussed here in expressjs/discussions#251, we are keeping docs only in the website. It is easier to maintain a single place instead. This PR will make sure that anything that is in the wiki but not in the website is being added.

New files

  • New features in 3x
  • Migrating from 2x to 3x
  • New features in 4x
  • Migrating from 3x to 4x

Notes

  1. There's pages for Migrating from 2x to 3x and New features in 3x in Wiki. However, 3x is already deprecated. So, I just added these files and referenced in en/3x/api.md.
  2. Added the Other modules from wiki home to the en/resources/utils.md.
  3. Only focused on en as of now.

Linked Issues

Edit: Rewrote description in order to give a better idea. Added few links so that relevant things can be navigated to from here.

Copy link

netlify bot commented Aug 10, 2024

Deploy Preview for expressjscom-preview ready!

Name Link
🔨 Latest commit 53032d1
🔍 Latest deploy log https://app.netlify.com/sites/expressjscom-preview/deploys/675b5a0d1c4e1e00081b1db0
😎 Deploy Preview https://deploy-preview-1565--expressjscom-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@IamLizu
Copy link
Member Author

IamLizu commented Aug 10, 2024

Before I proceed with other contents, I have a few questions,

  1. I have written these two files in markdown. Other guides such as migrating-4 or migrating-5 uses HTML. Can we be lose on this one or should I rewrite?
  2. I should add for other languages as well or en is okay for now? Perhaps we could include everything in en first, then move to other languages in different PR?
  3. Is this okay so far?
  4. Do we need to include this https://github.com/expressjs/express/wiki/ESON-JSON-Configuration?

cc: @wesleytodd @UlisesGascon

@bjohansebas
Copy link
Member

It would be better if the new pages were translated in different PRs, as it could happen like in https://expressjs.com/ru/resources/applications.html where the page is not translated into the corresponding language.

@IamLizu IamLizu force-pushed the content-migration branch from c9cffb1 to bb564a9 Compare August 11, 2024 17:38
@UlisesGascon
Copy link
Member

  1. I have written these two files in markdown. Other guides such as migrating-4 or migrating-5 uses HTML. Can we be lose on this one or should I rewrite?

I think that markdown is fine, but you might require html if you need additional features that are not included in the markdown support.

  1. I should add for other languages as well or en is okay for now? Perhaps we could include everything in en first, then move to other languages in different PR?

Don't worry, we can focus on English only in this PR.

  1. Is this okay so far?

Yeah! Thanks for leading this :)

  1. Do we need to include this https://github.com/expressjs/express/wiki/ESON-JSON-Configuration?

I don't think so, seems outdated in 2024 context 👍

@IamLizu
Copy link
Member Author

IamLizu commented Aug 13, 2024

Hi 👋

Regarding the items which were included as Pending in the PR description,

  1. Template Engines: It was just linked to website, so nothing to add here.
  2. Frameworks built on top of Express: Links to website, no other content, it seems it was removed from the website intentionally at some point or moved into different page. In any case, seems nothing to do here.
  3. Boilerplate: Most of them are old, but added those in the Express examples page.

And with that, I am marking this PR available for review 🤚

Edit: Forced pushed after a rebase in order to update the translation workflow file.

@IamLizu IamLizu marked this pull request as ready for review August 13, 2024 16:17
@IamLizu IamLizu changed the title Adding contents from Wiki of expressjs:express in website Adding contents from Wiki of expressjs:express in en language Aug 13, 2024
@IamLizu IamLizu requested a review from bjohansebas August 13, 2024 16:18
- migrating from 3x to 4x
- new features in 4x
- new features of 4x in migrating 4
- migrating from 3x to 4x in migrating 4
- migrating from 3x to 4x in new features of 4x
@IamLizu IamLizu force-pushed the content-migration branch from 3fb0708 to e79bffe Compare August 13, 2024 16:26
@wesleytodd
Copy link
Member

I am generally 👍 on this, but just want to call out there is a bunch of outdated info in this content from long ago. So it might be that we never even want to translate much of this and should take some time to clean it up first.

@IamLizu
Copy link
Member Author

IamLizu commented Aug 22, 2024

I am generally 👍 on this, but just want to call out there is a bunch of outdated info in this content from long ago. So it might be that we never even want to translate much of this and should take some time to clean it up first.

Make sense. However, these outdated contents are in their respective file such as "migrating to 3x". Those information are supposed to be outdated but should be kept for historical reason in my opinion.

Readers coming to the website will definitely read the new contents and I assume no is looking to migrate to 3.x for example, so we can safely ignore their translation as well maybe?

@IamLizu IamLizu requested a review from crandmck August 23, 2024 07:18
@bjohansebas
Copy link
Member

Is it useful to bring in the boilerplate when Express@5 is almost ready?

If it's for historical purposes, it might be better to handle it, for example, with a URL like /v4/examples and add the boilerplate there while keeping the main examples page as it was. What do you think about this idea?

@crandmck
Copy link
Member

Sorry I'm coming to this discussion late, as I've been out on vacation for a few weeks.

I think we should be cautious about what material we port from the wiki to the doc site, because:

  • Every page in the docs potentially adds work for maintenance and translation.
  • Our focus should be on 5.x docs. We have a LOT of work to do there, without taking on additional efforts that have less value.

Let's think carefully about whether it's worth investing the time and effort to move/maintain/translate these topics:

New features in 3x

"New" since when, 2014? How much value does this really have for real-world users?

Migrating from 2x to 3x

If someone (who?) still has a v2 app, do we really want to encourage them to migrate to v3? Wouldn't it make more sense to just rewrite it for v5, or even v4? This seems to me to have little or no real world value.

New features in 4x
Migrating from 3x to 4x

Again, this stuff is from around 10 years ago. The info should be covered in https://expressjs.com/en/guide/migrating-4.html but again, what's the best practice for someone (presumably not that many ppl) with a 3.x app? Is it to move to 4.x? Or rewrite it for 5.x? If we really want to spend time documenting how to upgrade from 3x -> 4x, then we should just update https://expressjs.com/en/guide/migrating-4.html.

Also re this comment

information are supposed to be outdated but should be kept for historical reason

IMHO, we should consider keeping the wiki around specifically for outdated historical/archival info. It's far less visible than the website, and does't ever need to be translated. These pages should clearly be marked as "OUTDATED ARCHIVED". Yes, we can move them to the website, but is that really how we want to spend our precious time and effort?

@IamLizu
Copy link
Member Author

IamLizu commented Sep 9, 2024

Hey @crandmck 👋

Sorry, I guess I also somehow missed this one.

I hear you, the new files are not new at all. But they are new in their own context. For example, in docs of 2.0, features of 3.0 are new. And that is where those said new features are mentioned, in their respective files (#1565 (comment)).

Funny thing I recently watched an conference talk of yours about how wiki should focus for developers and website for consumers. My bad, I would have at least approached a bit different way if I known better when we were discussing about deprecating the wiki in expressjs/discussions#251 where @wesleytodd and @UlisesGascon had a affirmative indication to what I said in expressjs/discussions#251 (comment).

That said, I think no one is going to read the docs for 2.0 or 3.0 so perhaps we can just simply ignore translation for those old files?

@crandmck
Copy link
Member

Yes, honestly, now that 5.0 is released, I would advocate for removing 2.x doc altogether, and perhaps even 3.x.

@IamLizu
Copy link
Member Author

IamLizu commented Oct 2, 2024

Yes, we could remove the old version(s) docs. However, I am being sentimental 😄

Personally, I would keep the old ones, maybe just ignore their translation but keep en for historical record.

cc: @expressjs/express-tc

@bjohansebas
Copy link
Member

Another option would be to route deprecated APIs through subdomains, as I suggest in #1556

@IamLizu
Copy link
Member Author

IamLizu commented Nov 6, 2024

What is our take on this?

@bjohansebas
Copy link
Member

After a long time since this PR was created,I'm not in favor of adding these resources to the page, for several of the reasons @crandmck mentioned.

Also, there are middlewares being added, and the policy since I've been here is not to add external resources, as it leads to discussions about what should and shouldn't be added. There are more important things to do here than debating whether to add such tools.

As @crandmck said, those wiki pages should be marked as "OUTDATED/ARCHIVED" This work can be done by a member of the triage team, as long as those permissions are given back to the triage team, since it seems they were disabled in the Express repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Deprecate the Express Legacy Wiki
6 participants