-
Notifications
You must be signed in to change notification settings - Fork 0
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
made model for Training Programs #5
Conversation
Models/TrainingPrograms.cs
Outdated
public string ProgramName { get; set; } | ||
public string MaxAttendees { get; set; } | ||
|
||
public List<TrainingPrograms> TrainingProgramsList = new List<TrainingPrograms>(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This won't need a list of itself inside itself.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But, it looks like it will need a list, of Employees who are enrolled in that program.
nashville-software-school/csharp-bangazonapi-boilerplate#9
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please adjust the list.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With the fixes, looks good
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me!
Purpose: create a model for Training Programs
How it fits in context of the project: establishes a get and set for the required information for this class to work.
Specific Feature affected:
How to test (Be thorough!): pull down to make sure that there are no conflicts. Check to see if you think all specified information is what you need to make this interaction work