Skip to content

assignment 1 completed#1

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

assignment 1 completed#1
kkimmins wants to merge 1 commit intomainfrom
assignment-one

Conversation

@kkimmins
Copy link
Copy Markdown
Owner

@kkimmins kkimmins commented Apr 1, 2026

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

I created the data structure flowchart, added all the SQL codes as prompted by the assignment, and wrote a short reflection piece after reading the provided article.

What did you learn from the changes you have made?

This was a very challenging and time-consuming assignment, but I was able to get a lot more familiar with SQL and was very useful.

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

There were a few different approaches to the SQL code that I wrote, such as using the AND instead of BETWEEN.

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

I actually lost a lot of my work because DB Browser for SQLite crashed and I forgot I couldn't just command+S to save (this is a habit of mine from working on apps that allows the shortcut)... I just redid them.

I also didn't know how to use INSERT INTO for the temp table exercise, so I had to Google that.

How were these changes tested?

The changes were tested by running the lines to verify proper output.

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

Checklist

  • I can confirm that my changes are working as intended

Copy link
Copy Markdown

@Dmytro-Bonislavskyi Dmytro-Bonislavskyi left a comment

Choose a reason for hiding this comment

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

Hi @kkimmins All looks fine, just A small clarification about diagram. I believe you mean like this:

Image

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