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

consoleになにも表示されないようにする #111

Merged
merged 13 commits into from
Mar 21, 2024

Conversation

nose221834
Copy link
Collaborator

@nose221834 nose221834 commented Mar 19, 2024

目的

consoleが本番環境で表示されないようにしたい

概要

  • 各ファイルのconsole.logを削除
  • useEffectの読み込みが行われる前読み込もうとして表示されるエラー抹消
  • fourth_placeのタイポを修正(ついで)

確認項目

  • ゲームをプレイ中にconsoleになにも表示されないか

項目追加

  • docker compose up で実行するとconsole.logが表示される
  • docker compose -f docker-compose.prod.yml upで実行するとconsoleになにも表示されない

不安・相談

@AobaIwaki123
Copy link
Collaborator

console.errorはあっていいの?

@AobaIwaki123
Copy link
Collaborator

AobaIwaki123 commented Mar 19, 2024

useEffectの読み込みが行われる前読み込もうとして表示されるエラー抹消

これはどうやって確認したらいい?

Copy link
Collaborator

@AobaIwaki123 AobaIwaki123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

動作確認OK

@AobaIwaki123
Copy link
Collaborator

useEffectの読み込みが行われる前読み込もうとして表示されるエラー抹消

確認できた

@nose221834 nose221834 requested a review from AobaIwaki123 March 20, 2024 15:48
@nose221834
Copy link
Collaborator Author

項目追加しました
console.logを削除するんじゃなくて、本番環境でビルドするときに削除されるようにしてみた
https://zenn.dev/aiq_dev/articles/dc19e087dda652

@AobaIwaki123 AobaIwaki123 merged commit 7b299b0 into develop Mar 21, 2024
1 check passed
@AobaIwaki123 AobaIwaki123 deleted the fix/nose/delete-console.log branch March 21, 2024 04:42
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.

2 participants