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

Add Visitors Book project #1103

Closed
wants to merge 1 commit into from

Conversation

anton-marchenko
Copy link
Contributor

@anton-marchenko anton-marchenko commented Feb 4, 2020

Add information about Visitors Book (open-source project that uses Express).

Proof link about Express usage.

This project was written by TDD methodology (and has 100% test coverage). Because of this, the project is important for community as study material.

Add information about Visitors Book (open-source project that uses Express).
@anton-marchenko
Copy link
Contributor Author

Waiting for #1094

@wesleytodd
Copy link
Member

I am hoping we can move most of this content off of the website sometime and into something more like an "awesome list". It is tough to maintain the content we have now, and I just don't want to make the situation worse. It is nothing against this example, but I am in general turning this kind of changes away.

@wesleytodd wesleytodd closed this Feb 15, 2020
@anton-marchenko
Copy link
Contributor Author

anton-marchenko commented Feb 15, 2020

I am hoping we can move most of this content off of the website sometime and into something more like an "awesome list". It is tough to maintain the content we have now, and I just don't want to make the situation worse. It is nothing against this example, but I am in general turning this kind of changes away.

But what about this description:
image

We need to delete this description in this case.

@wesleytodd
Copy link
Member

Sorry! I totally understand how that is miss-leading. It is not our intention, but much of the content on the website was written many years ago. Thanks for opening that pr!

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.

2 participants