search

copetit / whereareyou

for our family
1 stars 1 forks source link

postingを削除できるようにする #108

Closed copetit closed 2 years ago

copetit commented 2 years ago

やること

124 modalが完成されたら着手

成功：削除ボタン　→ パスワード確認　→ 一致したら　削除後のMain PageへRedirect 失敗：削除ボタン　→ パスワード確認　→ 一致しなかったら　エラーMSGをInput下に。

参考URL

shhwan commented 2 years ago

レビューお願いします

やったこと

確認方法

  1. http://localhost:3000/wau/posting ページでPostingを登録します
  2. Detail画面で間違ってるPWを入力して問題ないか　（エラーMSG出るか確認） 스크린샷 2022-03-02 00 28 13
  3. 正しいPWを入力すればMainにRedirectされて見ていたPostingが削除されること

確認して欲しいこと

copetit commented 2 years ago

確認しました。

動作には問題ありませんでした。 [MUST] パスワード入力後にenterを押すとgetパラメータで渡されます。enterで値が渡るのを防ぐ必要がありそうです。 少し調べましたが、あまり役に立ちそうなのは見つけませんでした。 inputにsubmitを使うとenterでも送信されるらしいけど、、onSubmit()のせいなのかな。

[NITS] 前に削除ボタンを押したら確認画面が出るようにしようと、いう話がありましたが、issueには書いてないのかな。 あと削除したら、すぐメイン画面に行くより、削除しましたも表示させるのもいい気もするのでこれらはまとめて別issueでやりましょう。pw削除フローをより親切にするとかのタイトルで issue作ってもらえますか？ (個人的にmodalもコンポーネント化したいな)

shhwan commented 2 years ago

ありがとうございます。 確認しました。

MUST

修正しました https://github.com/copetit/whereareyou/pull/139/commits/2ebdd9df297ef7d7ebce30c8cc89956e076b4070 Posting確認して、Enterで登録できてなかったため、合わせてできるようにしました。

間違ったPWを入力したらエラーになる。 正しいPWを入力したら削除＋転移されます。

NITSについて

確かに話してた気がします、、issueに書いてなくてすっかり忘れていました。 https://github.com/copetit/whereareyou/issues/140 issue作成しました。 追加案で入力画面で下にCheckBox作ってそれにチェックしたら進めることができる←追加しました。

shhwan commented 2 years ago

ちょっと負ける気もしますけど、Enter禁止しました。 後でProjectの最後に時間あったら直そうと思ってます。 https://github.com/copetit/whereareyou/pull/139/commits/481eef621d25efe8c50566b68593e9e540dfbe41

copetit commented 2 years ago

頑張ってもらったけど残念だったな。でもEnterは無しでも全然問題ないのでまた時間がある時でいいと思います。 以下動作確認しました。問題なかったのでマージしました。