Skip to content

Conversation

@NMai-source
Copy link
Collaborator

  • 対応する Issue が Linked Issues に設定されていることを確認してください

対応内容

・設定画面への遷移をナビゲーションに変更する
・上記修正に伴い、初期表示処理のタイミングを修正

スクリーンショット

before after
simulator_screenshot_4CF8D872-F5AB-4831-A2BA-44BFDFF65609 simulator_screenshot_A3640E61-90CD-4D1F-864A-09EA1607371F

close #8

@ykws
Copy link
Owner

ykws commented Dec 1, 2021

CI がエラーになっていたので re-run

Copy link
Owner

@ykws ykws left a comment

Choose a reason for hiding this comment

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

以下の表示となっているときは、 PR の branch が分岐した時点の main とマージしようとしている main に差異があるため、 rebase を行う必要があります。

Screen Shot 2021-12-01 at 23 22 22

@ykws
Copy link
Owner

ykws commented Dec 1, 2021

CI がエラーになっていたので re-run

rebase していないので test.yml がなくてエラーになっている

@NMai-source
Copy link
Collaborator Author

@ykws
rebaseを行いましたが、まだエラーが出ている状態です。
引き続き調査します。

@ykws
Copy link
Owner

ykws commented Dec 2, 2021

@NMai-source すみません、 test.yml があるのにエラーになっているので、 test の問題の可能性があります。こちらでも調査してみます。

@NMai-source
Copy link
Collaborator Author

@ykws
承知致しました。
よろしくお願い致します。

@ykws
Copy link
Owner

ykws commented Dec 2, 2021

@NMai-source #37 の対応を入れてみたので、再度お手数かけて申し訳ないですが、 rebase をお願いします。

#27 では CI パスするので問題が解消できている想定です。

@ykws ykws added the update branch This branch is out-of-date with the base branch label Dec 2, 2021
@NMai-source
Copy link
Collaborator Author

@ykws
承知いたしました。
ご対応頂きありがとうございます。
明日以降に確認させて頂きます。

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

Labels

update branch This branch is out-of-date with the base branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

設定画面への遷移をナビゲーションに変更する

2 participants