It looks great man! but I would change something it was me...I would add rxjava on data module with api. Therefore, it would be exposed to app module and to data module...I would do that because rxjava does not need an Android context.