Skip to content

Assignment one#1

Open
dtolgay wants to merge 8 commits intomainfrom
assignment-one
Open

Assignment one#1
dtolgay wants to merge 8 commits intomainfrom
assignment-one

Conversation

@dtolgay
Copy link
Copy Markdown
Owner

@dtolgay dtolgay commented Nov 4, 2025

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

Drawing relational schematic between two tables. I added necessary coding commands for basic table manipulation for SQL lite.

What did you learn from the changes you have made?

I learned the basic SQL lite operations like select, merge, filter, aggregate. Furthermore, I learned to create table.

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

For creating table, I hard coded the customer_id. There should be a better way of doing it, allowing database to assign an id to user.

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

Syntax is kinda hard. The commands are predictable but where to put paranthesis and commas is confusing.

How were these changes tested?

I tested code by copying and pasting into the DB browser for SQLite.

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

Checklist

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

Copy link
Copy Markdown

@anjali-deshpande-hub anjali-deshpande-hub left a comment

Choose a reason for hiding this comment

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

Can you please include the .pdf file for Section 1 of the assignment instead of .drawio file, for the review ?
Other parts of the assignment look fine.

@dtolgay
Copy link
Copy Markdown
Owner Author

dtolgay commented Nov 10, 2025

Done!

Copy link
Copy Markdown

@anjali-deshpande-hub anjali-deshpande-hub left a comment

Choose a reason for hiding this comment

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

Thanks for including the .pdf. I have reviewed it now. Well done 30/30

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