Skip to content

Assignment 1 submission#1

Open
a7bagher wants to merge 1 commit intomainfrom
assignment-one
Open

Assignment 1 submission#1
a7bagher wants to merge 1 commit intomainfrom
assignment-one

Conversation

@a7bagher
Copy link
Owner

@a7bagher a7bagher commented Apr 20, 2025

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

I made changes to the assignment1.sql file to complete assignment 1. I also added a PDF file for section 1 of the assignment.

What did you learn from the changes you have made?

learned a lot about working with tables in sql.

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

N/A

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

lots of challenges with getting the right syntax for sql. I overcame it by using google to learn more and using the class live code notes as a guidline.

How were these changes tested?

using sqlite execute function.

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

N/A

Checklist

  • I can confirm that my changes are working as intended

Copy link

@l-mack l-mack left a comment

Choose a reason for hiding this comment

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

Please review the feedback on your assignment below. Based on the comments provided, please revise and resubmit the indicated questions:

  • Logical Model: Please ensure the columns for which the relationship exists are indicated clearly and the direction of the relationship is is correct.
  • WHERE, Question 2: Have you correctly included two conditions using AND here?

Copy link

@l-mack l-mack left a comment

Choose a reason for hiding this comment

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

Great work, 29/30.

Some general feedback:

  • Your logical model is great, it contains table names, column names, relationship-type, it just lacks a clear relationships between columns and correct direction for the relationship.

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