Moved week 7,8,9 from the JavaScript repo to this repository (So JavaScript3). #1
Moved week 7,8,9 from the JavaScript repo to this repository (So JavaScript3). #1mkruijt merged 7 commits intoHackYourFuture:masterfrom mkruijt:master
Conversation
Merge branch 'master' of github.com:mkruijt/JavaScript3
GeorgeSapkin
left a comment
There was a problem hiding this comment.
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.
Why are these and other links relative? If it's in another repo use the full URL.
README.md
Outdated
| >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.
Or maybe HackYourFuture JavaScript 3 instead?
|
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. |
remarcmij
left a comment
There was a problem hiding this comment.
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.