Skip to content

Luis F. Salazar #141

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

lfsalazarcruz
Copy link

No description provided.

luis salazar added 5 commits June 28, 2018 12:26
Some methods are still missing.

Created the first Instructor object.
Created six different objects (2 of each).

Console.log different properties of each method. Everything is working correctly.

Still working on the last two methods.
Finished the debugsCode method and the grade method.
return `${this.name} announces to ${channel}, @channel standy times!`;
}
debugsCode () { // STILL MISSING STUDENT OBJECT ========================================
debugsCode (student.name, subject) { // STILL MISSING STUDENT OBJECT ========================================

Choose a reason for hiding this comment

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

Hey, Luis! Great job on this assignment! Most everything is working however on line 60 you have a small error. If you refer to line 26 I think you might spot the issue. Your code was very easy to read. Your comments were awesome!!

Added a grade property to the Student Constructor function.

Created a method for the Instructor Constructor function that substracts a random number from the student object.
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.

2 participants