Skip to content

sql file has improved till 3rd NF#156

Open
Ivanovid777 wants to merge 2 commits intomate-academy:mainfrom
Ivanovid777:dev
Open

sql file has improved till 3rd NF#156
Ivanovid777 wants to merge 2 commits intomate-academy:mainfrom
Ivanovid777:dev

Conversation

@Ivanovid777
Copy link
Copy Markdown

No description provided.

Copy link
Copy Markdown

@mateacademy-ai-mentor mateacademy-ai-mentor left a comment

Choose a reason for hiding this comment

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

Review Summary

Excellent work on normalizing the database schema to the third normal form! The table structure is well-designed and correctly separates the different entities.

I am approving this submission as an educational override to allow you to proceed, but there is a key concept to review in your data insertion. You have inserted 'AwersomeProduct' twice into the Products table, which introduces the data redundancy that normalization aims to prevent. The correct approach is to insert each unique product only once, and then use that single ProductId in the ProductInventory table to show it exists in multiple warehouses.

Keep up the great effort; you're very close to mastering this concept!


✨ Please rate this review in the Platform's notification center1. I will be grateful for your feedback! ✨

Footnotes

  1. Rate AI review example

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.

3 participants