update画面を作成する

[copetit]

やること

• /wau/updateでpositingファイルをコピーしてファイルを作成する
• getPostingById APIを読んで、値がフォームに埋め込む

参考URL

[copetit]

やったこと

• updateページを追加、固定でid 10番のpostingを表示しているので、自分の環境にあったidに変えて試してください。
• updateページに戻るボタン作成

思っただけの点

  const fileOneChange = (event: React.ChangeEvent<HTMLInputElement>) => {
......

ここらへんですが、 違うのはsetFileOneだけなので、callback関数として渡せばリファクタリングできるかもと、ふと思いました。

気になること

• プルリクにぼやきました。
• パスワードはどうしようかな。今のupdateは埋めてないです。hash化したのは元に戻せるか？ 面倒な作業になったらpwは更新できないように一旦フォームを削除しようかなと思ったりしました。

確認方法

http://localhost:3000/wau/update

• [ ] 以下の種類のpostingをアップロードしてみて、swaggerのAPI結果とupdateページを比較し、問題なく表示されるか確認 postingの切り替えはここのidを修正

  useEffect(() => {
  // TODO ここは更新モーダルからidを受けるるようにする
  // 今はid,10番のものを固定で表示しています
  getPosting(10).then((res) => {
    const { lat, lng } = res.locationinfo;
    const imgsFullUrl: string[] = [];

• 写真が一枚のposting

• 複数のposting

• 性別を不明ではないもの(デフォルトではなくてもちゃんと反映されるかを確かめるため)

• [ ] 戻るボタンでメイン画面に戻ること

[shhwan]

確認しました

思っただけの点

確かにPostingとUpdateはちょっと綺麗にできそうですね

気になること

• コメントしました
• そうですね、、 入力があるとき更新するか、そもそも更新できないようにするから考えてみるべきか悩みますね。 それは更新できるようにするチケットやりながら、判断してそっちで処理します。

確認しました。

１番〜３番変えながら確認して問題ありませんでした 写真一枚/複数 性別など全部問題ありませんでした。

すみません、、多分自分が理解を追ってない？ものだと思いますが ASKあります。

[copetit]

IMO修正、ASK返事しましました。

[shhwan]

ありがとうございます。 マージしました。