hypothesis / hypothesis/lms

Move `roles` off the `user` table

Open
#3,639 1 comment 0 reactions 0 assignees View on GitHub
refactoring Spike
Dominant language
Python
Stars
53
Forks
16
Avg merge
14d 5h
Merged PRs (30d)
14

Description

The `user` table includes a `rows` column (instructor, learner, admin, etc):

https://github.com/hypothesis/lms/blob/0ae07d059298b7b42c72614f1dd04e39078660fb/lms/models/user.py#L37-L38

This isn't right: the same user can be an instructor in one course and a learner in another.

We should move the `roles` column off the `user` table, probably onto the `group_membership` table.

Questions:

* Can users also have different roles in different Blackboard or Canvas groups within the same course?

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.