AdamMc331 / AdamMc331/MVWTF2024

question

Open
#12 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Kotlin
Stars
4
Forks
1
PR merge metrics
No merged PRs in 30d

Description

Thank you very much for your insights! I came across your article on droidcon titled "MVWT: Demystifying Architecture Patterns" and had a question regarding MVI.

Could we merge processEvent with the effect without needing TaskListSideEffects? like this

`data class MVITaskListViewState(isLoading,list,error)`
```
private fun processEvent(
currentState: MVITaskListViewState,
event: TaskListStateUpdateEvent,
) {
when (event) {
is TaskListStateUpdateEvent.RetryButtonClicked-> {
stateState(isLoading = true,emptylist(),error =null))
fetchTasks()
stateState(isLoading = false,error =null))
}
}
}
```

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.