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

生理画面の改修 #714

Merged
merged 16 commits into from
Oct 28, 2022
Merged

生理画面の改修 #714

merged 16 commits into from
Oct 28, 2022

Conversation

bannzai
Copy link
Owner

@bannzai bannzai commented Oct 26, 2022

Abstract

以下の内容の修正

  • 今回分の整理情報があれば表示する
  • 平均生理の計算方法が間違っていたので修正

Why

Links

Checked

  • Analyticsのログを入れたか
  • Navigator.of(context).pop() の後にContextを使用したメソッドを実行していない
  • 境界値に対してのUnitTestを書いた
  • パターン分岐が発生するWidgetに対してWidgetTestを書いた
  • リリースノートを追加した

@bannzai
Copy link
Owner Author

bannzai commented Oct 26, 2022

平均計算方法だけ修正すると混乱するだろうからバックアップ用のViewもしばらく出すようにしよう。そう考えると修正は小さい方が良いので、今回部分の表示の件 については後回しにしつつ最低限の変更点をまとめたPRを別途作る。一旦Draft

@bannzai bannzai marked this pull request as draft October 26, 2022 12:46
@bannzai bannzai marked this pull request as ready for review October 26, 2022 13:12
@bannzai
Copy link
Owner Author

bannzai commented Oct 26, 2022

やっぱり混乱するかどうかもわからないし、大した問題じゃない気もしてきたのでバックアップ案はなしで。計算は治った

@bannzai bannzai merged commit aba582a into main Oct 28, 2022
@bannzai bannzai deleted the fix/view/menstruation branch October 28, 2022 09:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant