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

管理者・メンター・アドバイザー・卒業生でログインしているときは休会リンクを非表示にした #5804

Conversation

sadanora
Copy link
Contributor

@sadanora sadanora commented Nov 17, 2022

Issue

概要

管理者・メンター・アドバイザー・卒業生でログインしているときは休会リンクが表示されないようにしました。

変更確認方法

  1. ブランチfeature/hide-a-hibernating-link-when-login-as-admin-mentor-adviser-graduatedをローカルに取り込む
  2. bin/rails sでローカル環境を立ち上げる
  3. まず休会リンクが表示されるケースを確認
    • hatsunoでログイン1し、右上の「Me」からドロップダウンメニューを表示。休会リンクがあることを確認する
  4. 次に休会リンクが表示されないケースを確認
    1. 管理者(adminonly)でログインし、右上の「Me」からドロップダウンメニューを表示。休会リンクがないことを確認する
    2. メンター(mentormentaro)でログインし、右上の「Me」からドロップダウンメニューを表示。休会リンクがないことを確認する
    3. アドバイザー(advijirou)でログインし、右上の「Me」からドロップダウンメニューを表示。休会リンクがないことを確認する
    4. 卒業生(sotugyou)でログインし、右上の「Me」からドロップダウンメニューを表示。休会リンクがないことを確認する

変更前

  • 管理者(adminonly)
    before_adminonly
  • メンター(mentormentaro)
    before_mentormentaro
  • アドバイザー(advijirou)
    before_advijirou
  • 卒業生(sotugyou)
    before_sotugyou

変更後

  • 管理者(adminonly)
    after_adminonly
  • メンター(mentormentaro)
    after_mentormentaro
  • アドバイザー(advijirou)
    after_advijirou
  • 卒業生(sotugyou)
    after_sotugyou

Footnotes

  1. 管理者・メンター・アドバイザー・卒業生以外のユーザーであればなんでも構いません。

@sadanora sadanora marked this pull request as ready for review November 17, 2022 04:54
@sadanora
Copy link
Contributor Author

@akingo55
おつかれさまです!
こちらのissueのレビューをお願いしたいのですが、ご都合いかがでしょうか?
ご検討よろしくお願いいたします〜:pray:

@akingo55
Copy link
Contributor

akingo55 commented Nov 20, 2022

@sadanora 遅くなりすみません!大丈夫です🙆‍♀️
問題なさそうです!

@sadanora
Copy link
Contributor Author

@akingo55
ご確認ありがとうございます!

お忙しい中レビューいただきありがとうございました〜!

@sadanora
Copy link
Contributor Author

@komagata
おつかれさまです!
メンバーレビューにてapproveいただきましたので、レビューをお願いできますでしょうか。
よろしくお願いいたします!

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 57b0a72 into main Nov 21, 2022
@komagata komagata deleted the feature/hide-a-hibernating-link-when-login-as-admin-mentor-adviser-graduated branch November 21, 2022 13:27
@github-actions github-actions bot mentioned this pull request Nov 21, 2022
18 tasks
@sadanora
Copy link
Contributor Author

レビューありがとうございました〜!

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