Skip to content
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

Ian - Kopi Tracker #165

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

Ian - Kopi Tracker #165

wants to merge 8 commits into from

Conversation

ianshalom
Copy link

Deliverable Submission

Please describe your comfort and completeness levels before submitting. Comfort Level (1-5):
2
Completeness Level (1-5):
2
How many hours did you spend on this assignment?
3
What part did you spend the most time on?
Trying to display the farms associated with the Kopi.
Does the assignment run without errors? If the assignment has errors, what are they?
I am unable to delete an instance of kopi. Oddly enough, I am able to delete the farms.
Describe one part of the code or choose one line you would improve upon. What would you improve?

What did you think of this deliverable? Any other comments?

@ianshalom
Copy link
Author

Please describe your comfort and completeness levels before submitting. Comfort Level (1-5):
2
Completeness Level (1-5):
2
How many hours did you spend on this assignment?
2
What part did you spend the most time on?
Trying to get the associated coffees with the farm to display accordingly.
Does the assignment run without errors? If the assignment has errors, what are they?
Fixed the delete for coffee but the format for my display is out of sorts.
Describe one part of the code or choose one line you would improve upon. What would you improve?

What did you think of this deliverable? Any other comments?
Good CRUD practice but furthers at this point in time seem too challenging for me.

@ianshalom
Copy link
Author

Please describe your comfort and completeness levels before submitting. Comfort Level (1-5):
2
Completeness Level (1-5):
2
How many hours did you spend on this assignment?
2
What part did you spend the most time on?
Managed to establish the M-M association and display kopis belonging to specific customers. I am however unable to add kopis to a customer during the creation of a customer with existing kopis displayed through the check box.
Does the assignment run without errors? If the assignment has errors, what are they?
No errors.
Describe one part of the code or choose one line you would improve upon. What would you improve?

What did you think of this deliverable? Any other comments

@ianshalom
Copy link
Author

Please describe your comfort and completeness levels before submitting. Comfort Level (1-5):
2
Completeness Level (1-5):
2
How many hours did you spend on this assignment?
3
What part did you spend the most time on?
Trying to add bootstrap to the assignment. It took me forever. I managed to add validation to the forms for customer and kopi creation.
Does the assignment run without errors? If the assignment has errors, what are they?
Unable to display kopi roast. Works in terminal but not in browser.

Describe one part of the code or choose one line you would improve upon. What would you improve?

What did you think of this deliverable? Any other comments

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