Cape Town | 26-ITP-Sept | Vaidah Simenti | Sprint 2 | Complete Sprint 2 Coursework - #1564
Vaidah-dev wants to merge 6 commits into
Conversation
✅ Deploy Preview for cyf-onboarding-module ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Vaidah-dev
added
the
Needs Review
JaypeeLan
added
Review in progress
There was a problem hiding this comment.
I don't think you really did a breakdown. Try to do more breakdown of the expression on line 4.
There was a problem hiding this comment.
Updated — I've added a step-by-step breakdown of the expression. Thanks
There was a problem hiding this comment.
There is another way to add comments in javscript. Kind sstate both.
There was a problem hiding this comment.
added the other way to add comments in JavaScript
There was a problem hiding this comment.
Line 4 should throw an error. Run the code and see why.
There was a problem hiding this comment.
removed console.log that i had added because it would not run anyway because of the TypeError
There was a problem hiding this comment.
You stated why there is an error, but did not state the error.
There was a problem hiding this comment.
can you state why you needed to rename the variable names for the code to run?
There was a problem hiding this comment.
added the reason for the renaming.
JaypeeLan
added
Reviewed
Vaidah-dev
added
the
Needs Review
2 participants
Self checklist
Task code
CYF-1039
Changelist
Completed all Sprint 2 exercises
Removed unwanted files from PR