FalckJoshua / DIT113-System-Development-Locus-Imperium

Other
0 stars 0 forks source link

Added a new font and changed the number of people display #8 - [merged] #78

Closed FalckJoshua closed 7 months ago

FalckJoshua commented 1 year ago

In GitLab by @andriid on Apr 18, 2023, 19:37

Merges 8-display-number-of-people-device -> main

What does this MR do?

Displaying the current amount of people in the room on the terminal screen.

Added a screen with number of people and a new font.

Related issues

Related to #1 #9 #14 #16 #17 #21 Closes #8

Acceptance criteria met

FalckJoshua commented 1 year ago

In GitLab by @andriid on Apr 18, 2023, 19:37

requested review from @vasilena

FalckJoshua commented 1 year ago

In GitLab by @vasilena on Apr 18, 2023, 19:41

Everything looks okay but pls add a reference for the fonts. Other than that I have nothing else to say. :smiley: :thumbsup_tone1:

FalckJoshua commented 1 year ago

In GitLab by @andriid on Apr 18, 2023, 19:41

added 1 commit

Compare with previous version

FalckJoshua commented 1 year ago

In GitLab by @vasilena on Apr 18, 2023, 19:42

approved this merge request

FalckJoshua commented 1 year ago

In GitLab by @andriid on Apr 18, 2023, 19:42

Thank you so much for the feedback! :kiss_mm:

FalckJoshua commented 1 year ago

In GitLab by @vasilena on Apr 18, 2023, 19:42

mentioned in commit 6498e3ac48dcdeb51d935a300176ca626c02e556