Skip to content

Azhar tunr_express#161

Open
mdazharaw wants to merge 26 commits into
wdi-sg:masterfrom
mdazharaw:master
Open

Azhar tunr_express#161
mdazharaw wants to merge 26 commits into
wdi-sg:masterfrom
mdazharaw:master

Conversation

@mdazharaw

@mdazharaw mdazharaw commented Apr 17, 2020

Copy link
Copy Markdown

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?
3
What part did you spend the most time on?
Part 1 done, Part 2 Restful Routing Delete still in progress
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?
Work in progress
What did you think of this deliverable? Any other comments?


Updated 20 Apr 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?
8
What part did you spend the most time on?
Writing up the working SQL queries and manipulating id of different tables
Does the assignment run without errors? If the assignment has errors, what are they?
Error when loading new playlist, first 2 playlists are working but beyond that other playlists not accessible, possibly due to id issues.
Describe one part of the code or choose one line you would improve upon. What would you improve?
Restructure code to work with ids given from table and not reassign id to rows.
What did you think of this deliverable? Any other comments?
Challenging

@mdazharaw

Copy link
Copy Markdown
Author

Updated 21 April 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
What part did you spend the most time on?
Added visit counter, furthers in progress
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?
Furthers, work in progress
What did you think of this deliverable? Any other comments?

@mdazharaw

Copy link
Copy Markdown
Author

Updated 22 Apr 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?
2
What part did you spend the most time on?
Reading up on login cookies and cookies in general to better understand its functions
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?
Havent implemented route blocking when not logged in and error messages for invalid login attempts
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