Skip to content

Conversation

@magdazelena
Copy link
Contributor

After leading Database module in December, I got some thoughts and feedback from the trainees. Below my proposed changes:

  1. Adding DBeaver screenshots
    Reasons:
    • I cannot install DBeaver on my company laptop and I need to show UI to the group during the session
    • Some buttons need to be documented and explained, like difference between run query buttons
  2. Adding JOIN to main session material
    Reasons:
    • Assignment requires use of JOIN
    • Trainees specifically requested JOIN being explained
    • SQL covered in session is very simple and majority of the group were done quickly with the excercises. There is time during the session to cover JOIN in my opinion.
  3. Explaing aliases in advanced material
    Reasons:
  • Again, specifically requested by trainees
  • They come across it often browsing through external materials.

Copy link
Contributor

@adamblanchard adamblanchard left a comment

Choose a reason for hiding this comment

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

Amazing, thanks a lot @magdazelena for taking your learnings and feedback from the session into tangible improvements here 🤩

They all look and sound great to me!

Two additional changes needed:

  1. Add the new dbeaver-ui.md to /SUMMARY.md so we can display it nicely on program.hackyourfuture.dk (otherwise it will just link to the raw github page)
  2. Because of that, we should rename "assets" to "session-materials", to match the naming pattern of our other modules, while we are here and making the url "public".

I don't know if you've worked with gitbook much before, but i'm happy to approve your PR and create a new one for those gitbook changes myself, if you'd like. Let me know!

- JOIN links only matching rows based on a condition\*.
- Good idea is to JOIN on the **foreign key**!

\*_there are different types of JOIN. [Read more different types of JOIN in this article (out of scope)](https://www.w3schools.com/sql/sql_join.asp)._
Copy link
Contributor

Choose a reason for hiding this comment

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

Really nice to call this out

@magdazelena
Copy link
Contributor Author

@adamblanchard made the requested changes, please check out :)

Copy link
Contributor

@adamblanchard adamblanchard left a comment

Choose a reason for hiding this comment

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

Perfect!

@adamblanchard adamblanchard merged commit 6d8f1e6 into main Jan 20, 2026
3 checks passed
@adamblanchard adamblanchard deleted the chore/feedbakc-from-db branch January 20, 2026 07:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

3 participants