-
Notifications
You must be signed in to change notification settings - Fork 112
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 route demo #11
Comments
removed in commit 14f5e3a |
Please explain why this was in this lesson and why you removed it. Thanks! |
@gaand No idea why it was in the lesson, so I removed it. Seemed out of scope. |
On second thought, I think it's better removed. This is covered very in depth in routing-static |
Okay. I just want to make sure that before you remove something form a lesson, you've thought about why it might be there. For instance, does covering it briefly here support covering it in more depth later? I don't know, but it might be worth thinking through. |
I agree with @gaand. Please take a bit longer to explain your thoughts in PRs and issues. It helps me since I'm not in the room and not always a part of the conversations you have. |
Seems unnecessary.
The text was updated successfully, but these errors were encountered: