Hirotaka-Hanai commented 1 month ago

チケット

92

msato-ok commented 3 weeks ago

@Hirotaka-Hanai pullして実行してみたところエラーが出ていて、動作確認ができていないので、解消してもらって、再度、レビューとさせてください。

Hirotaka-Hanai commented 2 weeks ago

@msato-ok ローカルでエラーを再現できなかったので、エラー内容からそれっぽい修正を行い、ログを追加しました。お手数ですが、これで動作確認を行っていただき、再度エラーがでるようでしたらもう一度ログを共有いただけないでしょうか？よろしくお願いします。

msato-ok commented 2 weeks ago

@Hirotaka-Hanai

@msato-ok ローカルでエラーを再現できなかったので、エラー内容からそれっぽい修正を行い、ログを追加しました。お手数ですが、これで動作確認を行っていただき、再度エラーがでるようでしたらもう一度ログを共有いただけないでしょうか？よろしくお願いします。

msato-ok commented 2 weeks ago

@Hirotaka-Hanai 設定画面のエラーは出なくなりました。DBを削除しなくても、エラーが出なくなったので、デフォルト値が無かったことが原因のようです。

ただし、まだ問題が2つ出ています。

Hirotaka-Hanai commented 2 weeks ago

@msato-ok 設定項目が追加された場合でも、デフォルト値が反映されるように修正をいれました。この状態でもう一度動作確認をお願いします。

msato-ok commented 2 weeks ago

@Hirotaka-Hanai 最新を pull したら、2つの問題ともに、修正されていました。

msato-ok commented 2 weeks ago

@Hirotaka-Hanai レビューしました。1つだけ、対応お願いします。 https://github.com/minr-dev/desktop/pull/134#discussion_r1658130070

Hirotaka-Hanai commented 2 weeks ago

@msato-ok コメント追加しました。

msato-ok commented 2 weeks ago

@Hirotaka-Hanai

LGTM

スカッシュして、マージしてください。マージ済みとなったブランチは削除お願いします。