Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UseCase + ViewModel #22

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

UseCase + ViewModel #22

wants to merge 3 commits into from

Conversation

StephaneBg
Copy link
Collaborator

No description provided.

@jtouzy
Copy link
Owner

jtouzy commented Feb 6, 2020

Il y a un peu trop de modifications au niveau du common, ça change trop l'archi.
Le UseCase et l'Interactor ?

@StephaneBg
Copy link
Collaborator Author

C'est juste du renommage. On trouve souvent ces notations en Clean Architecture côté Android. Cela évite en plus le suffixe Impl.

@jtouzy
Copy link
Owner

jtouzy commented Feb 6, 2020

Justement, l'architecture commence à un peu trop pencher côté Android.
Elle est prévue pour être générique pour multiplateforme et on se retrouve à devoir changer pour faire des spécifications particulières pour Android, notamment concernant le problème de rotation, juste pour éviter un re-draw de l'écran.

Je pense qu'il va falloir chercher une autre solution.
En plus, la PR impacte plusieurs choses : les tests d'un côté et cette implémentation de VM de l'autre côté, et j'ai du mal à voir les impacts de chaque ajout.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants