Fixed system diagram #20 - [merged]

[FalckJoshua]

In GitLab by @vasilena on Apr 17, 2023, 16:23

Merges 20-fix-the-system-diagram -> main

We added the "Wio Terminal" component to the diagram to show how the sensors communicate with the Broker. We then connected them with the Wio Terminal, which we connected to the Broker. The updated diagram now accurately represents all parts of our system and how they interact with each other.

[FalckJoshua]

In GitLab by @vasilena on Apr 17, 2023, 16:23

requested review from @philippi

[FalckJoshua]

In GitLab by @vasilena on Apr 17, 2023, 16:25

@philippi @shariqs Please review and approve/disapprove if there are still issues so we know how to proceed.

[FalckJoshua]

In GitLab by @philippi on Apr 17, 2023, 20:58

Commented on 9692a6d58c9b22f3b13a866a153ea6eeb3d55e52

The publish and subscribe syntax only makes sense between the app with the broker and between the WIO terminal and the broker. The sensors do not publish or subscribe to topics. Maybe add a third type of arrow indicating data transfer, other than the mqtt payload, might be a better representation of your system. What do you think?

[FalckJoshua]

In GitLab by @vasilena on Apr 18, 2023, 14:22

added 1 commit

• c686b530 - Fixed issues mentioned by TA in the diagram #20

Compare with previous version

[FalckJoshua]

In GitLab by @vasilena on Apr 18, 2023, 14:24

Commented on 9692a6d58c9b22f3b13a866a153ea6eeb3d55e52

Is this better? :blush: @philippi

[FalckJoshua]

In GitLab by @philippi on Apr 18, 2023, 20:14

Commented on 9692a6d58c9b22f3b13a866a153ea6eeb3d55e52

Yes, great job! Merge request approved.

[FalckJoshua]

In GitLab by @philippi on Apr 18, 2023, 20:15

approved this merge request

[FalckJoshua]

In GitLab by @vasilena on Apr 18, 2023, 20:16

mentioned in commit 53c9b88ebcfc6dbf651332e24bcab24856ed02e1