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

Feature reviews listing #2

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ashrafatef843
Copy link

@ashrafatef843 ashrafatef843 commented Feb 19, 2024

Solution

is a sample for clean architecture based on MVI architecture pattern that the UI depend on a single immutable state that demonstrates the screen.

Decision

The dealing with architecture patten depend on states idea is so important and adding more concrete structure more than MVVM as:

  • The ui depend on single source of truth (only one state) instead of many channel that emitting the data/events.
  • The ui starting with initial state which decrease the possibilities of handling null references and missing data.
  • Every state depend on the previous state so no state come out of out of nowhere. This is so good as the screen can be demonstrated through finite state machine and have fixed immutable state any time.

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