search

commew / timelogger-web

時間記録アプリ
https://timmew.commew.net
MIT License
2 stars 0 forks source link

LPページ追加 #136

Closed ai-ura closed 11 months ago

ai-ura commented 11 months ago

issueURL

91

この PR で対応する範囲 / この PR で対応しない範囲

Storybook の URL、 スクリーンショット

なし

変更点概要

レビュアーに重点的にチェックして欲しい点

コーディング内容は確認済みです。 リリースをお願いいたします。

補足情報

特にありません。

vercel[bot] commented 11 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
timelogger-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 12, 2023 8:02am
ai-ura commented 11 months ago

@keitakn

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

ファイルの追加とチェッカーの範囲外になる修正をしてアップしたのですが、 まだ next.config.js の設定ができていないので、全部終わった時点でご連絡します。

keitakn commented 11 months ago

@ai-ura 承知しました！ありがとうございます！

keitakn commented 11 months ago

@ai-ura @y-uchiida

ちょっと自分のローカルで試してみたのですが next.config.js を以下のようにすると / でやっても rewrites が有効にならないのですが https://github.com/commew/timelogger-web/blob/main/src/pages/index.tsx を削除したら問題なく表示されました！

どうもpages配下に index.tsx があるとそっちが優先して表示されるっぽいですね。

/** @type {import('next').NextConfig} */
const nextConfig = {
  reactStrictMode: true,
  rewrites: async () => {
    return [
      {
        source: '/',
        destination: '/index.html',
      },
    ];
  },
}

module.exports = nextConfig
スクリーンショット 2023-10-12 16 43 38

なので https://github.com/commew/timelogger-web/blob/main/src/pages/index.tsx を削除して next.config.js を私が提示した内容に変更して頂ければ本件の完了の定義は満たせると思います！

ai-ura commented 11 months ago

@keitakn @y-uchiida

ありがとうございます！！ 「connect ECONNREFUSED」というエラーが直せなくて悩んでいたのですが、 keitaさんに試していただいた方法で、私の方でもローカルで確認ができました。

next.config.js に設定を追加し、 src/pages/index.tsx ファイルを削除してPushしましたので、 改めてご確認をお願いいたします！

ai-ura commented 11 months ago

再三のご確認ありがとうございました。マージいたしました！

y-uchiida commented 11 months ago

本日不在にしていて、確認できずすみませんでした！ お二人ともご対応いただきありがとうございました！