Skip to content

Commit

Permalink
Model changes to add lms_user.lti_v13_user_id
Browse files Browse the repository at this point in the history
  • Loading branch information
marcospri committed Sep 11, 2024
1 parent cfae7b7 commit e70e673
Showing 1 changed file with 10 additions and 0 deletions.
10 changes: 10 additions & 0 deletions lms/models/lms_user.py
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,16 @@ class LMSUser(CreatedUpdatedMixin, Base):
lti_user_id: Mapped[str] = mapped_column(index=True)
"""ID of this user in the LMS, via LTI"""

lti_v13_user_id: Mapped[str | None] = mapped_column(sa.Unicode)
"""
The LTI1.3 ID of the user.
This will be often the same value as lti_user_id but it might be different in:
- LTI1.1 launches. Where this will be null.
- Upgraded instances from LTI1.1 where we prefer the LTI1.1 value (if exposed by the LMS).
In those cases lti_user_id will be the 1.1 value and we'll store here the 1.3 version.
"""

h_userid: Mapped[str] = mapped_column(unique=True, index=True)
"""The userid value in H. This is calculated hashing tool_consumer_instance_guid and lti_user_id together."""

Expand Down

0 comments on commit e70e673

Please sign in to comment.