Skip to content

Conversation

@rcantu1234
Copy link
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

Copy link

@stephsingler stephsingler left a comment

Choose a reason for hiding this comment

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

please submit a clean pull request with just space travel to mars file. You have 3 other projects attached to this PR

Copy link

@stephsingler stephsingler left a comment

Choose a reason for hiding this comment

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

Please submit a clean pull request before i can grade. There are 4 files in this PR and there should only be one.

Copy link

@reneemeyer reneemeyer 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 below. Please don't merge until you clean up this pull request. There are 4 changed files and there should only be one.


}
missionStatement() {
return this.ability;

Choose a reason for hiding this comment

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

this is almost correct, however it doesn't satisfy the tests, the ability is only returned if there is a crew member on the ship

this.ship = ship;
}
enterShip(ship) {
this.ship = ship;

Choose a reason for hiding this comment

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

the crew member needs to be entered into the ship's crew

return this.ship;
}
}
let mav = new Ship('Mars Ascent Vehicle', 'MAV', 'Ascend into low orbit');

Choose a reason for hiding this comment

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

const

}
}
let mav = new Ship('Mars Ascent Vehicle', 'MAV', 'Ascend into low orbit');
let hermes = new Ship('Hermes', 'Main Ship', 'Interplanetary Space Travel');

Choose a reason for hiding this comment

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

const

mav.crew.push(1);

// Created new object of CrewMember for test purposes.
let crewMember1 = new CrewMember('Rick Martinez', 'pilot', 'chemistry');

Choose a reason for hiding this comment

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

const

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.

5 participants