mako/coldcall_lti
1
0
Fork 0
Code Activity

Speak of the LMS generically, not Canvas specifically

Browse Source 
The tool targets any LTI 1.3 platform; Canvas is the primary target
but nothing outside the Canvas-specific custom variable substitutions
depends on it. User-facing strings and the README now say "LMS"
except where a mechanism genuinely is Canvas's (the $Canvas.* /
com.instructure.* substitutions and their handling), and the README
states plainly that the tool has so far been exercised only against
the saltire emulator, not yet a production LMS.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This commit is contained in:
Benjamin Mako Hill
2026-08-03 16:03:18 -07:00
parent cbbc890d44
commit 49d037692b
13 changed files with 52 additions and 42 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
coldcall_lti/models.py
View File

@@ -113,7 +113,7 @@ class Student(Base):
__tablename__ = "students"
id: Mapped[int] = mapped_column(primary_key=True)
# The LTI "sub" claim: stable, opaque, unique per Canvas user.
# The LTI "sub" claim: stable, opaque, unique per LMS user.
canvas_user_id: Mapped[str] = mapped_column(String(255), unique=True)
name: Mapped[str | None] = mapped_column(String(255))
sortable_name: Mapped[str | None] = mapped_column(String(255))