Skip to content

Update assignment2.sql#4

Open
nadeli11 wants to merge 2 commits intomainfrom
assignment-two
Open

Update assignment2.sql#4
nadeli11 wants to merge 2 commits intomainfrom
assignment-two

Conversation

@nadeli11
Copy link
Owner

@nadeli11 nadeli11 commented Apr 27, 2025

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

I added scripts as requested for Assignment 2.

What did you learn from the changes you have made?

One good thing I learned from SQL for Dummies book is that I can set an alias for referring to a table and use that in the script like when we right FROM vendor_inventory vi we can use vi instead of vendor_inventory and that makes the script shorter and cleaner. This is same in SQlite too.

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

There certainly always more than one approach but I didn't think about different ones.

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

The hint were really good to solve the challenging ones.

How were these changes tested?

I tested them in SQLite db and checked the outputs by displaying the results.

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

Checklist

  • I can confirm that my changes are working as intended

Copy link

@JzfL JzfL left a comment

Choose a reason for hiding this comment

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

Good SQL! please try complete section 1 of assignment 2

Copy link

@JzfL JzfL left a comment

Choose a reason for hiding this comment

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

OK I see, your assignment 2 ERD was already in main branch, good job
However, for future DSI modules as well as at future work place, please have your PR contain all the changes

@nadeli11
Copy link
Owner Author

nadeli11 commented Apr 30, 2025 via email

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