lastaflute / lastaflute-example-harbor

example project for LastaFlute as single project (with H2)
Apache License 2.0
6 stars 7 forks source link

退会のレイアウト崩れと不具合の修正 #9

Closed zoosm3 closed 2 years ago

zoosm3 commented 2 years ago

対応したこと

dbflute-example-on-springbootにWithdrawalを作ろうとして、harborのレイアウト崩れと不具合を見つけたので修正しました

修正前

スクリーンショット 2022-01-30 20 54 03 スクリーンショット 2022-02-19 20 30 39

修正後

スクリーンショット 2022-02-23 14 34 38 スクリーンショット 2022-02-23 14 34 52
zoosm3 commented 2 years ago

修正していませんが、気になる点がまだあるのでおつたえしておきます

  1. 退会理由 画面に区分値のコード SIT で表示されていますが、プルダウンでは名称の方が自然だと思います
  2. 退会理由の長さが0文字以上3文字以下とのことですが、byeくらいしか書けないのでもう少し長くできると思います
スクリーンショット 2022-02-21 21 51 08
jflute commented 2 years ago

おお、LastaFluteのプロジェクトの方まで本当に助かります。

退会理由 画面に区分値のコード SIT で表示されていますが、プルダウンでは名称の方が自然だと思います

これホントですね笑。 恐らく新人の方のチャレンジでやってもらったときの名残なので、どこかのタイミングで直そうかなと思います。

退会理由の長さが0文字以上3文字以下とのことですが、byeくらいしか書けないのでもう少し長くできると思います

もはやネタですね苦笑。多少、Exampleとして色々なケースを表現するためにあえていい加減にしてるところがありますが、こちらもどうにかします。

他のActionだと、そういう場合は #simple_for_example ってコメントで記してるので、せめてそれを入れるってところですね。

@Length(max = 10) // #simple_for_example just for validtion example
public String productName;

本題の修正はOKなのでマージします。 ありがとうございますー!