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

卒業生の就職希望者を就職希望者のスコープに追加 #4202

Merged
merged 2 commits into from
Feb 16, 2022

Conversation

kaisumi
Copy link
Contributor

@kaisumi kaisumi commented Feb 10, 2022

Issue

変更内容

  • userモデルのjob_seekersを、現役生と卒業生の就職希望者に拡張した。
  • テストとDBのフィクスチャにユーザーsotugyou-with-jobを追加し(talkも追加した)、就職済の卒業生とした。
  • 元々あったsotugyouは、就職希望がtrueなのに企業名が入力されている状態だったので、企業名などを削除した。
  • テストにもsotugyou-with-jobを追加し、sotugyou-with-jobは就職希望者の対象に含まれず、sotugyouが就職希望者の対象に含まれることを確認するようにした。

変更前

before.mov

変更後

after.mov

@kaisumi
Copy link
Contributor Author

kaisumi commented Feb 10, 2022

@kaisumi kaisumi marked this pull request as ready for review February 10, 2022 14:39
@kaisumi
Copy link
Contributor Author

kaisumi commented Feb 10, 2022

@garammasala29
お疲れ様です!こちらお手すきの時にレビューをお願いします 🙇

@kaisumi kaisumi self-assigned this Feb 10, 2022
Copy link
Contributor

@garammasala29 garammasala29 left a comment

Choose a reason for hiding this comment

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

@kaisumi さん
レビュー依頼ありがとうございます!
自分の手元でも動作確認ができました。新たなユーザーが作られていて確認しやすかったです😄
凄い細かな話になってしまうんですが、1点気になった箇所があります。
新たなユーザーの名前なんですが、初見でシゴトシタロウとなると求職中なのかな?と思い少し混乱してしまいました。
感覚的な問題だと思いますのでApproveにさせていただきます。よろしくお願いします!

@kaisumi
Copy link
Contributor Author

kaisumi commented Feb 12, 2022

@garammasala29
レビューありがとうございました!
確かに、「仕事し太郎」は「仕事したい」にも聞こえますね 🤔
「卒業 就職済美(ソツギョウ シュウショクズミ)」に修正しようと思います💡

@kaisumi
Copy link
Contributor Author

kaisumi commented Feb 12, 2022

@komagata
お疲れ様です!
こちらお手すきの時にレビューをお願いいたします 🙇

@kaisumi kaisumi requested a review from komagata February 12, 2022 11:33
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ですー🙆‍♂️

@machida machida force-pushed the feature/include_graduate_jobseekers_in_jobseekers_scope branch from 7568f9c to 577e750 Compare February 16, 2022 10:37
@machida machida merged commit a00f769 into main Feb 16, 2022
@machida machida deleted the feature/include_graduate_jobseekers_in_jobseekers_scope branch February 16, 2022 11:10
@github-actions github-actions bot mentioned this pull request Feb 16, 2022
54 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.

4 participants