copetit / whereareyou

for our family
1 stars 1 forks source link

細かい箇所を修正する #136

Closed copetit closed 2 years ago

copetit commented 2 years ago

やったこと

参考URL

mapsのconsole.logエラー対処方法 https://dev.classmethod.jp/articles/showing-marker-and-balloon-position-on-react-google-maps-api-map-app-in-a-nice-way/

copetit commented 2 years ago

やったこと

確認方法

shhwan commented 2 years ago

確認しました

favicon追加

問題なし (favicon svgもできるの初めて知りました)

titleの app name追加

コメントしました

mapsのconsole.logエラーが出ないように修正。(マップの読み込み前に実行されるとエラーになるらしい)

問題なし

postingにヘッダー追加

[IMO]index.htmlで持ってくるようですが、異なるようにしたいのであればHelmet使っても良さそうです。 個人的にはPostingのTitleもMainと同じでも全然問題ないと思います。

posting画面のパスワードについて説明文追加

🆒

メイン画面の登録ボタンにレスポンシブ追加。大画面だと大きめにしました。

良いと思います。ありがとうございます。

card infowindowの左側も丸くする(ダメだったらradiusを消して四角にする。) → 四角にしました。

以前話していたしこれで問題ないと思います。

slideページにmax-width指定、特徴・その他のところの両側に薄い線を入れて広く見えないようにしました。

これが全然安定感があって良いと思います。

他気になる点

Postingに必要ないConsole.logをしてる部分があって、もうここの削除お願いできますでしょうか? 以下のCOPY PASTEをお願いいたします。(テスト済み)

Posting.tsxの179行からですが

    try {
      await uploadFiles(data)
        .then((res) => res.data['imageUrl'])
        .then(async (res) => {
          await createPosting({
            PetName: petName,
            PetSex: petSex,
            PetAge: parseInt(petAge),
            PetInfo: petInfo,
            Detail: detail,
            LostDate: lostDate,
            Address: address,
            CreatedDate: `${nowYear}-${nowMonth}-${nowDate}`,
            UpdateDate: `${nowYear}-${nowMonth}-${nowDate}`,
            locationinfo: location,
            user: {
              Password: password,
              MailAddress: mailladdress,
            },
            contents: {
              imageUrl: res,
              // TODO: Video導入後編集
              videoUrl: '',
            },
          }).then(() => {
            window.location.href = '/wau';
          });
        });
    } catch (error) {
      console.log(error);
    }
  }

これで問題ないと思います。 182行あたりのConsole.logと204行あたりのConsole logを消しただけです。

shhwan commented 2 years ago

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