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

空の/buzzアクセス時の500エラーを修正 #6754

Merged
merged 5 commits into from
Aug 1, 2023

Conversation

djkazunoko
Copy link
Contributor

@djkazunoko djkazunoko commented Jul 22, 2023

@djkazunoko djkazunoko self-assigned this Jul 22, 2023
@djkazunoko djkazunoko marked this pull request as ready for review July 22, 2023 04:46
@djkazunoko djkazunoko requested a review from komagata July 22, 2023 04:46
@@ -23,6 +23,7 @@ def update

def set_buzz
@buzz = Buzz.first
@buzz ||= Buzz.create(body: 'test')
Copy link
Member

Choose a reason for hiding this comment

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

testという文字列が本番環境で出てくるということなのでこれは問題があると思います。

seedとdata-migrationに適切な初期データを入れるといいと思います。
適切な初期データは @machida さんに聞いてみてください。

@machida
Copy link
Member

machida commented Jul 26, 2023

@djkazunoko @komagata デザインを調整して、初期データも変更しましたー

@komagata
Copy link
Member

@machida @djkazunoko

緊急の対応として本番DBを直接いじって修正しておきました。
https://bootcamp.fjord.jp/buzz

それとは別にこちらのPRは進めていただければと思います〜。

@machida
Copy link
Member

machida commented Jul 26, 2023

@komagata

了解ですー👍

  • このPRで初期データの変更と、デザインの微調整が含まれています。
  • 初期データは本番に反映しなくても大丈夫です(直接本番環境に手動で入力するため)
  • デザイン調整は終わっているので、今週のデプロイでリリースできたらと思います。
  • /buzz を一般公開するのは一通り記事が揃ってからにしたいと思います(今は /buzz へのリンクはメンター、管理者しか見れない状態です)

@machida
Copy link
Member

machida commented Jul 26, 2023

@djkazunoko rubocop が通らないみたいで CI が取っていない状況です。こちら見ていただきたいですー

@djkazunoko
Copy link
Contributor Author

@machida
CI全てパスしました🙏

@komagata
data-migrationで初期データを作成しました。f343ef5
レビューお願いします。

Copy link
Member

@komagata komagata left a comment

Choose a reason for hiding this comment

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

確認させて頂きました。OKです〜🙆‍♂️

@komagata komagata merged commit 739b9f1 into main Aug 1, 2023
1 check passed
@komagata komagata deleted the hotfix/fix-empty-buzz-500-error branch August 1, 2023 04:22
@github-actions github-actions bot mentioned this pull request Aug 1, 2023
16 tasks
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.

3 participants