Skip to content

Conversation

@borjasolerme
Copy link
Owner

@borjasolerme borjasolerme requested a review from samarv January 24, 2019 16:01
Copy link
Collaborator

@samarv samarv left a comment

Choose a reason for hiding this comment

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

Project Objective

  • Convert all of your constructors into ES6 Classes using the class and extends keywords. You should be able to run your same logs and they should build out the proper expected behaviors.

  • Create classes for Lambda Instructor

  • Create classes for Lambda PM

  • Create classes for Lambda Students.

Stretch

  • Extend the functionality of the Student by adding a prop called grade and setting it equal to a number between 1-100.
  • Now that our students have a grade build out a method on the Instructor (this will be used by BOTH instructors and PM's) that will randomly add or subtract points to a student's grade. Math.random will help.
  • Add a graduate method to a student.
  • This method, when called, will check the grade of the student and see if they're ready to graduate from Lambda School
  • If the student's grade is above a 70% let them graduate! Otherswise go back to grading their assignments to increase their score.

Module Objective

  • describe and demonstrate how class syntax works.
  • convert a pseudo classically instantiated program into class syntax

Conclusion

Hey Borja, Your code is pretty spot on. Your git flow is great. Your approach is clean and precise.

There's just one thing I want to reiterate - Stretch. I am pretty sure, that you have got this all covered and you understand everything but this is easy stuff. You need to do the hard thing. You need to challenge yourself. You need to build those muscles.

Other than that, You are doing a marvellous job. Lemme know if you have any questions.

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.

3 participants