-
Notifications
You must be signed in to change notification settings - Fork 281
Moved week 7,8,9 from the JavaScript repo to this repository (So JavaScript3). #1
Conversation
Merge branch 'master' of github.com:mkruijt/JavaScript3
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM except for the links.
|1.|• [Object Oriented Programming and Classes](fundamentals/oop_classes.md)<br>• [The `this` keyword](fundamentals/this.md) |[Reading Week 7](/Week1/README.md)|[Homework Week 7](/Week1/MAKEME.md)| | ||
|2.|• Structure for a basic SPA (Single Page Application) <br>• [XMLHttpRequests](fundamentals/XMLHttpRequest.md) <br>• API calls|[Reading Week 8](/Week2/README.md)|[Homework Week 8](/Week2/MAKEME.md)| | ||
|3.|• [Promises](fundamentals/promises.md)<br> • (re)writing data structures (in JSON) <br>• Async vs Sync <br>• [Event Loop (order of execution)](fundamentals/event_loop.md) |[Reading Week 9](/Week3/README.md)|[Homework Week 9](/Week3/MAKEME.md)| | ||
|1.|• [Object Oriented Programming and Classes](../../../fundamentals/blob/master/fundamentals/oop_classes.md)<br>• [The `this` keyword](../../../fundamentals/blob/master/fundamentals/this.md) |[Reading Week 1](/Week1/README.md)|[Homework Week 1](/Week1/MAKEME.md)| |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why are these and other links relative? If it's in another repo use the full URL.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see comment in other pr
README.md
Outdated
@@ -1,14 +1,14 @@ | |||
>Please help us improve and share your feedback! If you find better tutorials or links, please share them by opening a Pull Request. | |||
|
|||
# HackYourJavaScript | |||
# JavaScript3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Or maybe HackYourFuture JavaScript 3
instead?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yep I like it.
All links seem to work fine. One thing that I sometimes found confusing is that the prereading for a lecture is to be found in the README/MAKEME of the previous lecture (hence the need for a Week 0). Can't we just have the prereading in the README of the corresponding lecture? |
"Can't we just have the prereading in the README of the corresponding lecture?" yes, we can do that I also have thought about removing the makeme entirely and puttting the homework and readings all in the readme. It could be that in some cases the file would become quite big. For the content wise remarks, could you please create an issue, I rather solve one problem at a time :) |
Oh and if i'm correct the error handeling is partly covered here: https://github.com/HackYourFuture/debugging, merging this repo with the javascript assignments is the next thing on my todo list 💃 |
|
Can we finalize and merge this one, please? I would like to make some further changes as I need to teach this module to class 14. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I need to make further changes here. Let's work with relative paths for now until we find a better solution.
Hi George, we have to work with this repo. Like Jim said, let's work with relative paths for now until we find a better solution. I'm going to merge this now
No description provided.