Skip to content

Assignment #1#1

Open
dacresl wants to merge 2 commits intoassignment-onefrom
main
Open

Assignment #1#1
dacresl wants to merge 2 commits intoassignment-onefrom
main

Conversation

@dacresl
Copy link
Copy Markdown
Owner

@dacresl dacresl commented Aug 11, 2025

Assignment1

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)

Adding code to run different queries relating to a Farmers Market.

What did you learn from the changes you have made?

How to run a variety of quires in SQL, as well as how to join tables.

Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?

No as I am not experienced enough.

Were there any challenges? If so, what issue(s) did you face? How did you overcome it?

I ran into many challenges and had to test code, try to fix it and re-run it.

How were these changes tested?

By running the code in SQL Lite to see if the output was correct or if there were any errors.

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

I had issues with JOIN.

Checklist

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

Copy link
Copy Markdown

@DwardEE DwardEE left a comment

Choose a reason for hiding this comment

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

Nice work on the logic model!

For the asignment, your SELECTs, WHERE (Q2 correct), JOIN ordering, and both aggregate queries meet the key requirements. The CASE logic is fine, though making the pepper flag case-insensitive would help catch all varaitions. Your temp table creation and insert are clear.

These are just minor fixes, so I'll mark your assignment as complete.

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.

2 participants