badoo / badoo/MVICore

Add initial state to Bootrstrapper invocation

Open
#92 0 comments 1 reaction 0 assignees View on GitHub
enhancement
Dominant language
Kotlin
Stars
1.3k
Forks
88
PR merge metrics
No merged PRs in 30d

Description

We need to decide what to do in `Bootstrapper` based in the initial state quite often. Right now this needs to be passed manually to implementation constructor, which is quite inconvenient when e.g deciding the initial state based on `TimeCapsule`, as a reference cannot be stored for it inside constructor.

Proposed solution is to change `Bootstrapper` and `BaseFeature` so that initial state is passed automatically to invocation.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reading the Bootstrapper and BaseFeature invocation paths, especially how initial state is currently passed to implementation constructors. Trace the TimeCapsule-related use case described in the issue. Done means initial state is passed automatically to invocation without manual constructor handling, with the affected behavior covered by the repository's existing tests.

Written by the indexing model from the issue text.

Assessment

Tech stack
kotlin
Domain
mobile-dev
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.