You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I noticed that we're investing a lot of time in supporting and testing for multiple-organizations per user.
This goes against the Tahoe 2.0 architecture of having one LMS organization per user.
Here's what we need to do:
Helpers should expect only one organization per user.
No multi-org helpers should be created.
Multi-org tests should be removed.
@shadinaif I know that we're re-using existing models which limit our freedom in changing them, but is this model fix possible?
class UserOrganizationMapping(models.Model):
- user = models.ForeignKey(settings.AUTH_USER_MODEL, ...)+ user = models.OneToOneField(settings.AUTH_USER_MODEL, ...)
organization = models.ForeignKey(Organization, ...)
Required test changes:
The helper tests below adds two organizations for a single user, this shouldn't be done:
I noticed that we're investing a lot of time in supporting and testing for multiple-organizations per user.
This goes against the Tahoe 2.0 architecture of having one LMS organization per user.
Here's what we need to do:
@shadinaif I know that we're re-using existing models which limit our freedom in changing them, but is this model fix possible?
Required test changes:
The helper tests below adds two organizations for a single user, this shouldn't be done:
tahoe-sites/tahoe_sites/tests/test_helpers.py
Lines 26 to 47 in aa58ddc
The text was updated successfully, but these errors were encountered: