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

Remove some community resources #1579

Merged

Conversation

bjohansebas
Copy link
Member

As we discussed here expressjs/discussions#258, some of the recommended resources on the page are no longer used by the community. Therefore, they will be removed so they are no longer recommended while we look for new options from a new community.

Notes:

  • I can only perform the removal in Spanish and English since those are the languages I am familiar with.

Copy link

netlify bot commented Aug 21, 2024

Deploy Preview for expressjscom-preview ready!

Name Link
🔨 Latest commit 82abb24
🔍 Latest deploy log https://app.netlify.com/sites/expressjscom-preview/deploys/66ce4824207d0a0008c8eedf
😎 Deploy Preview https://deploy-preview-1579--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.

Copy link
Member

@UlisesGascon UlisesGascon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Copy link
Member

@IamLizu IamLizu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am mostly okay with this. However, I would like "GitHub Discussion" be added as the community place for support.

@bjohansebas
Copy link
Member Author

@IamLizu If the other TC members agree, I’d be happy to add "GitHub Discussion" as a place for support.

@crandmck
Copy link
Member

Thanks for this @bjohansebas, I've been wanting to do this for some time!

Yes, please do add a link to https://github.com/expressjs/discussions.

I see in expressjs/discussions#258 there was discussion of adding the Slack channel. I'd vote in favor of that, but that can be done in another PR; no need to hold up this one.

I can only perform the removal in Spanish and English since those are the languages I am familiar with.

That's understandable, however, I'd like to ask that we try to keep all the langs in sync as much as possible. One does not need to read the language to delete the sections in question. Since the content is well-structured, simply do this:

  1. For Gitter, remove the section with the link to https://gitter.im/.. including the header.
  2. For Mailing list, remove the section with the link to https://groups.google.com/.. including the header.
  3. For IRC, remove the section with mention of #express and freenode including the header.

@bjohansebas Can you please try to do that? If you run into difficulty, please comment here and I'll try to help.

@bjohansebas
Copy link
Member Author

@crandmck I've noticed there are references to the wiki on GitHub. Should I also remove those links?

@crandmck
Copy link
Member

crandmck commented Sep 2, 2024

The only links to the wiki I see are in "Moving to Express 4". We can leave those for now. If/when we decide to delete the wiki, we should do a thorough review of the content there to determine what needs to move into the docs and what can be simply deleted.

@crandmck crandmck merged commit da0a7ab into expressjs:gh-pages Sep 2, 2024
6 checks passed
@bjohansebas bjohansebas deleted the remove-references-of-community branch September 7, 2024 20:11
chrisdel101 pushed a commit to chrisdel101/expressjs.com that referenced this pull request Nov 3, 2024
* remove some community resources

* remove some community resources in other langs

* add github discussions
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.

4 participants