Skip to content

Daniel - tunr express#166

Open
dannxt wants to merge 7 commits into
wdi-sg:masterfrom
dannxt:master
Open

Daniel - tunr express#166
dannxt wants to merge 7 commits into
wdi-sg:masterfrom
dannxt:master

Conversation

@dannxt

@dannxt dannxt commented Apr 17, 2020

Copy link
Copy Markdown

Deliverable Submission

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

Completeness Level (1-5):
Part 2 Playlist functionalities; 2

How many hours did you spend on this assignment?
6

What part did you spend the most time on?
Fixing bugs

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

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?

@dannxt

dannxt commented Apr 20, 2020

Copy link
Copy Markdown
Author

Deliverable Submission

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

Completeness Level (1-5):
2; cookie tracking and view counter

How many hours did you spend on this assignment?
3

What part did you spend the most time on?
Fixing bugs

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

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?

@dannxt

dannxt commented Apr 22, 2020

Copy link
Copy Markdown
Author

Deliverable Submission

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

Completeness Level (1-5):
2

How many hours did you spend on this assignment?
5

What part did you spend the most time on?
Trying to do conditional insert; add user only if user not already in table

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

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