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

Userモデルからexperienceカラムを削除 #8026

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

thmz337
Copy link
Contributor

@thmz337 thmz337 commented Aug 22, 2024

Issue

概要

#7906 の対応でexperiencesカラムを追加したことにより、experienceカラムが不要となったため、その削除を行なった。

変更確認方法

画面の変更などはありませんので、#7970 の動作確認と同様の確認を行って、不具合が発生しないことを確認していただきたいです。

  1. chore/remove-experience-column-from-userをローカルに取り込む
  2. bin/rails db:resetを実行
  3. foreman start -f Procfile.devでサーバーを立ち上げる
  4. hajime(現役生・研修生なら誰でもOK)でログイン
  5. 登録情報変更へ移動
  6. 「プログラミング経験」が複数選択可能なチェックボックス形式になっていることを確認
    (hajimeであれば「未経験」なのでチェックが入ってない状態で正しいです)
  7. 好きにチェックを変えて「更新する」をクリック
  8. komagata(メンターor管理者なら誰でもOK)でログイン
  9. さきほど登録情報を変更したユーザーのプロフィールページへ
  10. ユーザー非公開情報の「経験」の欄に先ほどチェックした選択肢が表示されていることを確認

※このブランチでbin/rails db:resetを実行した後、mainブランチへ戻り他の作業をする(元のデータベースに戻す)には以下を実行してください。
(migrateの履歴等を元に戻すためです)

  1. bin/rails db:rollbackを実行
  2. schema.rbが変更されてしまうので、それを戻す(git restore db/schema.rb)
  3. git checkout mainでmainブランチへ
  4. bin/rails db:resetを実行

@thmz337 thmz337 self-assigned this Aug 22, 2024
@thmz337 thmz337 marked this pull request as ready for review August 22, 2024 14:52
@thmz337
Copy link
Contributor Author

thmz337 commented Aug 22, 2024

@kyokucho1989

お疲れ様です。こちらのPRについてレビューをお願いできますでしょうか。
ご都合の悪い場合はおっしゃってください。

よろしくお願いします。

@kyokucho1989
Copy link
Contributor

@thmz337
問題ないですー
一週間ほどお待ちください!

Copy link
Contributor

@kyokucho1989 kyokucho1989 left a comment

Choose a reason for hiding this comment

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

@thmz337
動作確認できました。
問題ないのでapproveしますー

@thmz337
Copy link
Contributor Author

thmz337 commented Aug 25, 2024

@kyokucho1989
ありがとうございます!

@komagata
確認よろしくお願いいたします。

@thmz337 thmz337 requested a review from komagata August 25, 2024 15:03
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