Skip to content

Chelsea | CLI-Todo-SQL#104

Open
chelsejw wants to merge 11 commits intowdi-sg:masterfrom
chelsejw:master
Open

Chelsea | CLI-Todo-SQL#104
chelsejw wants to merge 11 commits intowdi-sg:masterfrom
chelsejw:master

Conversation

@chelsejw
Copy link

@chelsejw chelsejw commented Apr 15, 2020

Deliverable Submission

Please describe your comfort and completeness levels before submitting.

Comfort Level (1-5): 3

Completeness Level (1-5): 4

How many hours did you spend on this assignment? 4.5

What part did you spend the most time on?

  • In general Googling for SQL commands to do what I want to do.
  • Trying to get the minutes difference between two timestamps. Was stuck trying to use some SQL commands that I found out later doesn't work in PSQL.
  • Fixing my bug for getting average items added per day.

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

  • I think it runs without errors.

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

  • Code is pretty messy & repetitive. There are some things like getting the average (e.g. line 127) which might have been achievable using SQL commands.

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

  • Good practice to get you looking for more SQL commands. But I have to say I'm hating SQL more and more by the second.

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