oasis4hedev / salsa

Web app for creating a syllabus in an accessible HTML document, publishing to the Canvas LMS, and more.
https://syllabustool.com
Other
6 stars 5 forks source link

Hotfix/add setting redirect_by_lms_account_id #592

Closed opsaaaaa closed 4 years ago

opsaaaaa commented 4 years ago

before: Given a client had a remote account_id that did not match a local lms_account_id When they visited the document#course route. Then they would see a redirected to many times error.

now: There is a setting called redirect_by_lms_account_id

When it is off Then do not attempt to redirect to an organization by the lms_account_id. (we assume that their is correct)

When it is on Then attempt to redirect them with the correct using the lms_account_id to find the organization.

When it is on And we fail to find an existing organization with the account_id Then continue with the current organization. (most likely the root organization.)