FalckJoshua / DIT113-System-Development-Locus-Imperium

Other
0 stars 0 forks source link

Implemented #66 - [merged] #112

Closed FalckJoshua closed 7 months ago

FalckJoshua commented 1 year ago

In GitLab by @cjoshua on May 17, 2023, 16:52

Merges 66-update-the-amount-of-people-manually-app -> main

What does this MR do?

The plus and minus buttons on the home activity now adjust the people count by 1 increment and publish it to the broker.

Related issues

[Related to: #1 #8 #9 #14 #16 #17 #21 ]

Acceptance criteria

FalckJoshua commented 1 year ago

In GitLab by @cjoshua on May 17, 2023, 16:52

requested review from @carlosca

FalckJoshua commented 1 year ago

In GitLab by @carlosca on May 17, 2023, 17:07

added 8 commits

Compare with previous version

FalckJoshua commented 1 year ago

In GitLab by @carlosca on May 17, 2023, 17:37

Good job the code looks clean!

FalckJoshua commented 1 year ago

In GitLab by @carlosca on May 17, 2023, 17:37

approved this merge request

FalckJoshua commented 1 year ago

In GitLab by @carlosca on May 17, 2023, 17:37

mentioned in commit 8ddfd3679736a3a45745afb6279fce44f833de75

FalckJoshua commented 1 year ago

In GitLab by @carlosca on May 17, 2023, 17:45

I don't know why but the commit 9bb1e3f5 made by the issue assignee disappeared. ¯\(ツ)