Skip to content

Rpm/checkpoint2#7

Open
r-pm wants to merge 49 commits into
gh-pagesfrom
rpm/checkpoint2
Open

Rpm/checkpoint2#7
r-pm wants to merge 49 commits into
gh-pagesfrom
rpm/checkpoint2

Conversation

@r-pm

@r-pm r-pm commented May 17, 2020

Copy link
Copy Markdown
Owner

Checkpoint Rubric

This is the rubric that your instructor will use to grade your checkpoints. Please do not edit.

Checkpoint 1

  • All tests passed: 40 points
  • Proper use of documentation (commenting on code): 15 points
  • Properly indented code: 15 points
  • Demonstrated effective use of JavaScript: 30 points

Checkpoint 2

  • The application works as it should: 40 points
  • Proper use of documentation (commenting on code): 15 points
  • Properly indented code: 15 points
  • Demonstrated effective use of JavaScript and the DOM API: 30 points

Checkpoint 3

  • Use of React: 25 points
  • Accesses an API: 25 points
  • Proper use of documentation (commenting on code): 25 points
  • The application functions as it should: 25 points

r-pm and others added 30 commits April 18, 2020 18:18
…terminal app game & have the game solve itself
…but now error show up in html. error reads that start / end are not defined in html
…can now see my actual code errors. had to re-write all variables inside of event function to accomodate event handler. no more errors from js but css has gone to shit. next need to redo css so i can see the functionality of js
…o all towers - worried this will affect move functions
…ts. something is preventing the function from moving into movePiece
…ns or counts i expect. playing with [i], [j] give closer consol.logs, but not correct
… nested for loop because i wanted there to be continuity in the var solution when it had items removed. made var solution and array, but am now having passing tests that way. test does not recognize ['abcd'] as an iterable array?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant