Allow more flexible identifiers for LRS entries

Open on Drupal.org →

Created on 17 December 2024, 11 months ago

Problem/Motivation

lrs_xapi currently hard-codes entity id and user id in the schema https://git.drupalcode.org/project/lms_xapi/-/blob/1.0.x/lms_xapi.instal...

Discussed this with @graber and we think we could just use a string identifier instead, this would give us more flexibility to e.g. use the course status ID if we want to.

Steps to reproduce

Proposed resolution

Remove the entity ID and user ID columns and replace with a varchar.

Remaining tasks

User interface changes

API changes

Data model changes

📌 Task

Status

Active

Version

1.0

Component

Code

Created by

🇬🇧United Kingdom catch

Live updates comments and jobs are added and updated live.

Sign in to follow issues

Merge Requests

!2Allow more flexible identifiers for LRS entries
Merged
🇵🇱Poland Graber
updated 11 months ago

Comments & Activities

Issue created by @catch
First commit to issue fork.
Merge request !2Support more flexible LMS-LRS identifiers → (Merged) created by Graber
Comment 11 months ago →
🇵🇱Poland Graber
Comment 11 months ago →
🇬🇧United Kingdom catch
Not sure about an event here, tagged services might be easier to follow - left a comment on the MR.

The schema change and making this flexible seems good though.
Comment 11 months ago →
🇵🇱Poland Graber
Let's have a follow up with tagged services as I need to move to other things now. Fixed some bugs and merged.
Comment 10 months ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024