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

Add Sponsor ActionCreator test #718

Closed
takahirom opened this issue Feb 1, 2019 · 8 comments · Fixed by #751
Closed

Add Sponsor ActionCreator test #718

takahirom opened this issue Feb 1, 2019 · 8 comments · Fixed by #751

Comments

@takahirom
Copy link
Member

takahirom commented Feb 1, 2019

@takahirom takahirom changed the title Add Sponsor ActionCreator, Store tests Add Sponsor ActionCreator, Store test Feb 1, 2019
@takahirom takahirom changed the title Add Sponsor ActionCreator, Store test Add Sponsor ActionCreator test Feb 1, 2019
@xnxaxo
Copy link
Contributor

xnxaxo commented Feb 2, 2019

@tomoya0x00
Copy link
Contributor

Thanks! Assigned 👍

@xnxaxo
Copy link
Contributor

xnxaxo commented Feb 2, 2019

@tomoya0x00
環境の事で質問なのですが、forkしてcloneしてReadmeのRequirementsまで行ったのですが、
参考となるテストコードを開いた際に表示が真っ赤なのでテストを実行するとエラーになるのですが
行った手順以外に設定が必要なのでしょうか?
添付のような状態になっていて、アプリもビルドして端末に入るのですが、Home画面のDay1, Day2に何も表示されません。
申し訳ありませんが、わかりましたらお手数かけますがご回答のほどよろしくお願い致します.
2019-02-02 18 24 12

@takahirom
Copy link
Member Author

Can you check this?
#548

日本語で失礼します。
多分通信できてないのはエミュレータの環境のせいかもしれません。Google Playが入れられるエミュレーターでお試しください

@xnxaxo
Copy link
Contributor

xnxaxo commented Feb 2, 2019

@takahirom
ありがとうございます。
教えていただいた手順で再度、検証してみます。

@takahirom
Copy link
Member Author

If you have any question, kindly feel free to ask me!

@takahirom
Copy link
Member Author

Can you try this?

in settings.gradle

include ':frontend:android', //':frontend:ioscombined',

@xnxaxo
Copy link
Contributor

xnxaxo commented Feb 2, 2019

@takahirom
settings.gradle tried.
The following error has been resolved.
java.lang.Throwable: Module 'ioscombined_iOSMain' and module 'api-impl_iOSMain' have the same content root:

However, since it was raised by another issue, I removed it from the target.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants