Skip to content

Setting Multi Modules - #8

Merged
ho2ri2s merged 7 commits into
masterfrom
multi-module-setting
Mar 7, 2020
Merged

ho2ri2s merged 7 commits into
masterfrom
multi-module-setting

Conversation

@ho2ri2s

@ho2ri2s ho2ri2s commented Mar 7, 2020 •

Copy link
Copy Markdown
Contributor

Overview

add multi module

Detail

  • appはDIで解決すべきクラスがあるModuleを全てimplementationする必要がある
  • featuresは基本repositoryをimplementationする。
    • modelはrepositoryがapiで公開しているためimplementationしなくて良い。
  • repositoryはdbとapiモジュールをimplementationしている。
    • featureから直接apiを叩いたりdbアクセスするのではなく、repositoryを経由する。
  • modelは何も参照しない。
  • apiからはmodelを参照しない。自身のmodule内に作成するResponseのみを使用する。

76137487-f58d9b00-6080-11ea-835b-b6d6956fe2ef

Close Issue

#6

@ho2ri2s
ho2ri2s force-pushed the multi-module-setting branch from de90ebf to 710db1d Compare March 7, 2020 06:22
@ho2ri2s
ho2ri2s force-pushed the multi-module-setting branch from 710db1d to fb1a91a Compare March 7, 2020 06:48

@b4tchkn b4tchkn left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

http___lgtm

@ho2ri2s
ho2ri2s merged commit 4aad60e into master Mar 7, 2020
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