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

1.2 Layer your components, keep Express within its boundaries #393

Closed
bikingbadger opened this issue May 1, 2019 · 6 comments
Closed

1.2 Layer your components, keep Express within its boundaries #393

bikingbadger opened this issue May 1, 2019 · 6 comments
Labels

Comments

@bikingbadger
Copy link

First I want to say a huge thank you for this resource. As someone just starting out it's great to see the community coming together to provide this. I'm was going over the list and on the layering components (1.2), I get the overall picture but when clicking on more details there is just a gif. Is there anyway to give a bit more detail on this page? I see that there is quite a nice detailed answer in #252 , I don't mind summarizing that into a pull request and giving the people who answered credits.

@bikingbadger
Copy link
Author

I was a bit proactive and create a PR on what I was thinking.

@js-kyle
Copy link
Contributor

js-kyle commented May 6, 2019

@bikingbadger Thanks for the feedback + PR! We'll get that reviewed 👍

@delliriun
Copy link

A GIF/image is such a bad idea. How can blind people read this?
I lose one frame and need to see everything again.

@maliiiith
Copy link

I recently found this repo from a Reddit post and I have to say this has answered a lot of questions I had for sometime.

I agree with @delliriun on this. I experienced the same thing in Layer your app, keep Express within its boundaries's 1min explainer GIF.

@goldbergyoni
Copy link
Owner

@delliriun @malithrw A PR with better explanation is undergoing

@stale
Copy link

stale bot commented Sep 18, 2019

Hello there! 👋
This issue has gone silent. Eerily silent. ⏳
We currently close issues after 100 days of inactivity. It has been 90 days since the last update here.
If needed, you can keep it open by replying here.
Thanks for being a part of the Node.js Best Practices community! 💚

@stale stale bot added the stale label Sep 18, 2019
@stale stale bot closed this as completed Sep 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants