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

# Feature/196, 198, 199 Grouping users #201

Merged
merged 2 commits into from
Nov 15, 2017

Conversation

TatsuyaIse
Copy link
Contributor

  • Prepare group model.(methods are not complete) 「グループモデルの用意(グループ管理ページ向け検索メソッドのみ実装済み)」
  • Add group-model method to models index. 「グループモデルを、 /lib/models/index.js に追加」
  • Add group route to routes index. 「 /admin/groups パスでのグループクラスのメソッド呼び出しを、
    /lib/routes/index.js に追加」

* Prepare group model.(methods are not complete)
* Add group-model method to models index.
* Add group route to routes index.
@TatsuyaIse TatsuyaIse self-assigned this Nov 14, 2017
@yuki-takei
Copy link
Member

Group っていう名前、UserGroup の方がいいかなあ。

@yuki-takei
Copy link
Member

password ってどう使う想定?

UserGroup の管理ポリシーをどうするかも見えてこないな。管理者ユーザーはみんな自由に変更できるんだったらパスワード不要だし、グループの管理権限を用意するならユーザーロールとか必要だろうし。

@TatsuyaIse
Copy link
Contributor Author

Group っていう名前、UserGroup の方がいいかなあ。

自分もその方がよいかもとうすうす感じていたので、Group → UserGroup へ、モデル名変更しておきます。

Group っていう名前、UserGroup の方がいいかなあ。

UserGroup に関して編集をする際にパスワードを聞かれる感じを想定していたのですが、グループごとに個別のパスワードを用意するのではなく 管理者ユーザーはみんな自由に変更できる の方が妥当かつ管理が複雑化せずに済むと思うので、 GroupUser のパスワードに関連する記述は削除しようかと思います。

* Prepare group model.(Fix group model with reviewed feed back.)
* Fix routes and index for ↑.
@yuki-takei yuki-takei merged commit 5f36706 into feat/grouping-users Nov 15, 2017
@yuki-takei yuki-takei deleted the imprv/prepare-group-model branch November 15, 2017 08:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants