Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

⬆️: Upgrade to React 18 #2

Merged
merged 27 commits into from
Nov 17, 2023
Merged

⬆️: Upgrade to React 18 #2

merged 27 commits into from
Nov 17, 2023

Conversation

sinho-lee
Copy link
Contributor

@sinho-lee sinho-lee commented Nov 7, 2023

✅ What's done

Reactバージョンアップ対応

  • Reactのバージョンアップ v17.0.3 → v18.2.0 (対応Commit: 517b1de)
  • react-domのバージョンアップ v17.0.3 → v18.2.0 (対応Commit: ec4df26)
  • react-router-domのバージョンアップ v5.2.0 → v5.3.4 (対応Commit: f56df81)
    • v5.3.4に上げた理由はv5.3.3からはReact v18との互換性の問題を解決したので、v5.3系の最新バージョンにしました。詳細についてはOtherのreact-router-domのURL先に記載されています。
  • Reactのバージョンアップによるソースコード修正 (対応Commit: 7144e7f)
  • react-router-domのバージョンアップによるソースコード修正 (対応Commit: 7144e7f)
  • チャット送信時、送信済みの内容が2回表示される現象を解消 (対応Commit: f9f83b9)
  • QRCodeが2個表示される現象を解消 (対応Commit: 0506f12)
  • GENERATE_SOURCEMAPの設定を追加 (対応Commit: 565ebf3)

ライブラリのバージョンアップ対応

  • TypeScriptのバージョンアップ v4.2.3 → v4.9.5 (対応Commit: 1f3f762)
    • react-scriptsがTypeScriptのv5系をサポートしていないため、v4系の最新版に上げました。
  • TypeScriptのバージョンアップによるソースコード修正 (対応Commit: fd9a082)
  • react-scriptsのバージョンアップ v4.0.3 → v5.0.1 (対応Commit: e6117a3)

Tests

  • 各画面が表示されることを打鍵テストで確認
  • 各機能が問題なく動くことを打鍵テストで確認
  • npm run testコマンド実行
  • npm run lintコマンド実行

Other (messages to reviewers, concerns, etc.)

@creature-water-valley creature-water-valley requested review from tisurgm and removed request for creature-water-valley November 7, 2023 07:17
@tisurgm tisurgm changed the base branch from master to develop November 7, 2023 09:36
frontend/package.json Show resolved Hide resolved
frontend/src/index.tsx Outdated Show resolved Hide resolved
frontend/src/framework/logging/Logger.dev.ts Outdated Show resolved Hide resolved
frontend/src/framework/logging/Logger.prod.ts Outdated Show resolved Hide resolved
frontend/src/chat/components/Routing.tsx Outdated Show resolved Hide resolved
frontend/package.json Show resolved Hide resolved
@creature-water-valley creature-water-valley removed the request for review from tisurgm November 10, 2023 07:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants