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

Rachel - Kopi Tracker #156

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

Rachel - Kopi Tracker #156

wants to merge 15 commits into from

Conversation

Rachelik
Copy link

Deliverable Submission

Please describe your comfort and completeness levels before submitting. Comfort Level (1-5):
3

Completeness Level (1-5):
2

How many hours did you spend on this assignment?
4

What part did you spend the most time on?
Renaming the variables and figure out which one i have not rename...

Does the assignment run without errors? If the assignment has errors, what are they?
yes

Describe one part of the code or choose one line you would improve upon. What would you improve?
no edit and delete page yet. hardcode css for now.

What did you think of this deliverable? Any other comments?
helps me to get use to the syntax

@Rachelik
Copy link
Author

Rachelik commented May 13, 2020

Deliverable Submission
Part 2 (13 May 2020)

Please describe your comfort and completeness levels before submitting. Comfort Level (1-5):
3

Completeness Level (1-5):
2

How many hours did you spend on this assignment?
1 hr

What part did you spend the most time on?
Most of the 1 hr (logging and add edit/delete) not taking too much time (submitted part). Outside of that 1 hr (not submitted part), spent 3-4 hrs trying to edit the original kopis table from text to references for Roast. roast_id cannot be edited. Used change_column and finally drop table. Created new kopis table/coffees table and everything still referred back to the old table. Finally, dropped kopis and do coffees instead. Coffees was then not found.... do a series of destroy and generate. Probably much better off starting a new app than editing one column in a table.

Does the assignment run without errors? If the assignment has errors, what are they?
yes

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?
Think I have encountered all the the problems necessary, but there is always more that it makes me think I have not truly understand....

@Rachelik
Copy link
Author

Deliverable Submission
Part 3 (14 May 2020)

Please describe your comfort and completeness levels before submitting. Comfort Level (1-5):
3

Completeness Level (1-5):
2

How many hours did you spend on this assignment?
30 mins

What part did you spend the most time on?
Add customer model. (didn't include the M-M as I get error when I added the checkboxes.)

Does the assignment run without errors? If the assignment has errors, what are they?
yes

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

What did you think of this deliverable? Any other comments?
how to make m-m work.

@Rachelik
Copy link
Author

Rachelik commented May 15, 2020

Deliverable Submission
Part 3 (14 May 2020)

Please describe your comfort and completeness levels before submitting. Comfort Level (1-5):
3

Completeness Level (1-5):
3

How many hours did you spend on this assignment?
4 hrs (Added CSS + some correction to make m-m work and show).

What part did you spend the most time on?
finding way to style the checkboxes... the checkbox and label can get detached or squeezed together... copy and pasting bootstraps class across every file. As the content increased, the more tedious it gets copy-pasting class across all files.

Does the assignment run without errors? If the assignment has errors, what are they?
yes

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

What did you think of this deliverable? Any other comments?
CSS should be implemented with the first model before doing other models.

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