Skip to content

Conversation

@drdhennessy
Copy link
Owner

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
Draw a couple of ERD diagrams, write some sql code to answer the Qs in the assignment.

What did you learn from the changes you have made?
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
I tried several approaches - especiallly on the cross join. It worked but I wasn't convinced it was correct.

Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
Especially on the cross join and the last one on updating the table by adding current_quantity. I had to delete the table and start again a few times. Then the added column wasn't showing up. But no errors.

How were these changes tested?
In db browser for sqlite.

A reference to a related issue in your repository (if applicable)
N/A

Checklist
[x ] I can confirm that my changes are working as intended

Copy link

@efantinatti efantinatti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dear Dean Hennessy

Session 1:
1 - OK.

Session 2:
2 - OK.

Session 3:
3 - Partial.
-- Cross Join - Q1: Missing CROSS JOIN.
-- DELETE - Q1: You query deletes everything not the oldest entry.
-- UPDATE - Q1: COALLESCE not taking effect. nullvalues are seen.

Final:
4 - Check the aforementioned points prior to merging your feature branch 'assignment-two' to 'main'.

Mark: 64 points.

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.

4 participants