Sending message - Githubissues

CSC207-2022F-UofT / course-project-translate-chat-program

course-project-translate-chat-program created by GitHub Classroom

0 stars 4 forks source link

Sending message #55

Closed dannylu03 closed 1 year ago

dannylu03 commented 1 year ago

What I've Done:

Fixed the architecture for the Send Message use case so that the presenter only talks with the controller. And the hiearchical structure is clear.

This makes it such that the Send Message architecture adheres to clean architecture as the inner layers won't depend on the outer layers.

What I have left:

Unit tests for the send message functions
Documentation for Send Message use case

dannylu03 commented 1 year ago

You have a reference to DBService in ChatScreen and in MessageInteractor. ChatScreen should not have a reference to DBService at all, only presenters and controllers, while MessageInteractor should reference a gateway!

Nice catch, just made the changes