Skip to content

Assignment two#2

Open
moecui22 wants to merge 3 commits intomainfrom
assignment-two
Open

Assignment two#2
moecui22 wants to merge 3 commits intomainfrom
assignment-two

Conversation

@moecui22
Copy link
Owner

@moecui22 moecui22 commented Nov 8, 2025

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

Completed assignment2.sql with COALESCE, window functions, string manipulation, UNION, and data manipulation queries (INSERT/UPDATE/DELETE). Created two ER diagrams (ERD_Assignment2_Prompt1.png and ERD_Assignment2_Prompt2.png). Added written responses in Assignment2.md, including Section 1 (Prompt 3) and Section 4 (Ethics Reflection).

What did you learn from the changes you have made?

Designed logical data models using entity relationships and normalization principles.

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

I considered separating SQL scripts by topic (e.g., COALESCE, window functions), but decided to keep all sections in a single file for easier review and reproducibility.

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

For the ERDs, ensuring proper relationships between tables (e.g., employee–shift–order) required a few attempts before the model was consistent.

How were these changes tested?

All SQL queries were tested in DB Browser for SQLite

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

NA

Checklist

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

@moecui22 moecui22 self-assigned this Nov 8, 2025
Copy link

@niyazmnazari niyazmnazari left a comment

Choose a reason for hiding this comment

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

Thank you for submitting assignment 2. All tasks are correct.
Mark: 70/70

P.S. You missed the relation between tables and their cardinality.

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