Skip to content

Assignment one#1

Open
Kayessemm wants to merge 3 commits intomainfrom
assignment-one
Open

Assignment one#1
Kayessemm wants to merge 3 commits intomainfrom
assignment-one

Conversation

@Kayessemm
Copy link
Copy Markdown
Owner

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

I completed the logical model, SQL code, and written response.

What did you learn from the changes you have made?

I learned how to write some basic code. I had some trouble with the later prompts (7 onwards) but I ultimately figured them out with time.

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

None in particular.

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

I misunderstood my results for Query 7, so I spent too long rewriting and rerunning it, until I browsed the vendor_booth_assignments table and saw my initial results were actually correct. I also had trouble writing query 9 as one query until I reread the question and realized I was supposed to run two components separately.

How were these changes tested?

I mainly tested by browsing the data and spot-checking some counts/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
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 @Kayessemm, .sql part is done and diagram too, but response in section 4 of file Assignment-1.md is missing.

…nse in the .md. Sorry for missing that the first time!
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.

Approved

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