Skip to content
This repository was archived by the owner on May 14, 2024. It is now read-only.

Alternative readings, instead of MDM and SpeakinJS#54

Merged
M3kH merged 5 commits into
masterfrom
gijscor-patch-2
Sep 6, 2018
Merged

Alternative readings, instead of MDM and SpeakinJS#54
M3kH merged 5 commits into
masterfrom
gijscor-patch-2

Conversation

@gijscor
Copy link
Copy Markdown
Contributor

@gijscor gijscor commented Sep 4, 2018

Both resources are very far from beginner friendly and clearly suppose previous coding experiences.
Example of a sentence used in these resources:

"javaScript contains a standard library of objects, such as Array, Date, and Math, and a core set of language elements such as operators, control structures, and statements. Core JavaScript can be extended for a variety of purposes by supplementing it with additional objects"

As the level of English of some of our students is limited, and many have no background in programming, I think these resources are mostly just having students read information that they are not yet ready to process, and mostly will leave them confused. There are much better resources (that we have written ourselves) that explain it much better, and that we don't link to in the reading material of this week. (I have created a seperate pull request for this.

As replacement to MDM and speakingjs webpage(too compicated), I thought it would be a good idea to replace them with our own documentation on the topics we discuss that week. These links are now considered Bonus material, but I think it's good that are obliged to read up on the topics that will be discussed in class, so that the teachings actually stick better with the students.
Copy link
Copy Markdown
Contributor

@remarcmij remarcmij left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See comments.

Comment thread Week2/README.md Outdated
Comment thread Week2/README.md Outdated
Fixed list, based on @remarcmij 's suggestoin
removed the 2 links discussed
Comment thread Week2/README.md Outdated
- [Naming conventions](./../../../../fundamentals/blob/master/fundamentals/naming_conventions.md)
- Helpful resource: http://jsbooks.revolunet.com/ (here you can find tons of free JavaScript books online)

- The 50 best websites to Learn JavaScript: http://www.codeconquest.com/blog/top-50-websites-to-learn-javascript/
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is just confusing and doesn't provide any quality insight.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree, that was already in the original document though.

Copy link
Copy Markdown
Member

@M3kH M3kH left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Even if the title of this PR says Alternative Reading this PR doesn't provide any additional value.

I would rather close it.

remove unuseful resources
@gijscor
Copy link
Copy Markdown
Contributor Author

gijscor commented Sep 5, 2018

@M3kH ready to merge now?

@M3kH M3kH merged commit 780f40e into master Sep 6, 2018
@M3kH M3kH deleted the gijscor-patch-2 branch September 6, 2018 07:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants