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

vue.js を使っているところでも複数の role を認識出来るようにする #4214

Merged
merged 3 commits into from
Feb 16, 2022

Conversation

ot0m1
Copy link
Contributor

@ot0m1 ot0m1 commented Feb 13, 2022

ref: #4025

要件

#4025 から引用。

現在はユーザーに一つの role しか認識されていないので、
(currentUser.role == "mentor" || currentUser.role == "admin")
このように書いている。

本来であれば、komagata、machida は mentor と admin の 2つの role を持っているので、
currentUser.role == "admin" は不要。

ところが、mentor と admin の 2つの role を持っているのに、admin の方しか認識されていないので、
currentUser.role == "mentor" が true にならない。
そのため、 (currentUser.role == "mentor" || currentUser.role == "admin") と書いている。

複数の role が認識されるようになれば、
mentor と admin の 2つの role を持っている komagata、machida でも、
currentUser.role == "mentor" が true になり、
(currentUser.role == "mentor" || currentUser.role == "admin") と書かずに、
currentUser.role == "mentor" だけで済むようになる。

要はメンターと管理者が利用できればいい機能に対して、ログインしているアカウントの role が menter を含んでいるかを判断するようなコードに変更する。

現在 menter を含む role を持っているアカウントは admin のみ。

補足

ref: https://discord.com/channels/715806612824260640/809595476847493192/942035335154659328

上記のような背景からすで複数の role を認識できるようになっているとして、if 文の修正のみの対応を行う。

画面イメージ

なし。

確認方法

  1. feature/Identify-multiple-roles-even-when-using-vuejs ブランチをローカルに持ってくる(参考:https://qiita.com/great084/items/ad74dd064a2c2bc47cff
git fetch origin pull/{このプルリクエストのid}/head:{任意のブランチ名}
git checkout {上記でfetchした任意のブランチ名}
  1. 管理者(komagata or machida)のアカウントでログイン
  2. app/javascript/question-edit.vueapp/javascript/comments.vueapp/javascript/answer.vueのファイルで表示しているページにおいて、if 文が true となり対象の id や class の要素がレンダリングされていることを確認

@ot0m1 ot0m1 marked this pull request as ready for review February 14, 2022 12:53
@ot0m1
Copy link
Contributor Author

ot0m1 commented Feb 14, 2022

@garammasala29 お疲れ様ですレビューお願いいたします。
一部通らないテストがあったので、 #4203 を先取って取り込んでいます。

@ot0m1 ot0m1 requested a review from garammasala29 February 14, 2022 12:54
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.

@ot0m1 さん
お疲れ様です、レビュー依頼いただきありがとうございました😄
Vueファイルでのmentoradminが重複しているコードがなくなっていることを確認し、adminolyという管理者のみのユーザーで表示されないことも確認できました。
LGTMです!よろしくお願いします。

@ot0m1
Copy link
Contributor Author

ot0m1 commented Feb 15, 2022

@garammasala29 レビューありがとうございました〜!
@komagata レビューお願いいたします。

@ot0m1 ot0m1 requested a review from komagata February 15, 2022 13:54
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 7a3a8a4 into main Feb 16, 2022
@komagata komagata deleted the feature/Identify-multiple-roles-even-when-using-vuejs branch February 16, 2022 03:50
@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.

3 participants