Store custom_canvas_user_id as LMSUser.lms_api_user_id

hypothesis · Nov 29, 2024 · f1c2f70 · f1c2f70
1 parent 86c631b
commit f1c2f70
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/lms/services/user.py b/lms/services/user.py
@@ -75,6 +75,9 @@ def upsert_user(self, lti_user: LTIUser) -> User:
     def upsert_lms_user(self, user: User, lti_params: LTIParams) -> LMSUser:
         """Upsert LMSUser based on a User object."""
         self._db.flush()  # Make sure User has hit the DB on the current transaction
+
+        # API ID, only Canvas for now
+        api_id = lti_params.get("custom_canvas_user_id")
         lms_user = bulk_upsert(
             self._db,
             LMSUser,
@@ -101,6 +104,7 @@ def upsert_lms_user(self, user: User, lti_params: LTIParams) -> LMSUser:
                         text('"lms_user"."lti_v13_user_id"'),
                     ),
                 ),
+                "lms_api_user_id",
             ],
         ).one()
         bulk_upsert(