Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Store LTI1.3 assignment ID explicitly in the DB #6616

Merged
merged 1 commit into from
Sep 2, 2024

Conversation

marcospri
Copy link
Member

@marcospri marcospri commented Aug 29, 2024

For:

To talk to the Name and Roles LTI API we use the service URL sent in LTI1.3 courses.

However to filter that by assignment we need to use the LTI1.3 assignment ID on that same course URL.

The ID is sent in all launches but if present, we prefer the LTI1.1 (for backwards compatibility).

With this PR we now store explicitly the LTI1.3 ID to later be use the call the LTI API.

Testing

Nothing changes

The LTI1.3 is stored in the DB

  • Check the values in the DB:
docker compose exec postgres psql -U postgres -c "select  resource_link_id, lti_v13_resource_link_id from assignment;"                                                                                                                                                                                                                                   
             resource_link_id             |       lti_v13_resource_link_id       
------------------------------------------+--------------------------------------
 f330509e8ee8e465a55c4c88a4fcc9ef1ee992bd |  c2e8a1acbf6534de1db7d9dc65f034f69e87735c | acf313e2-e654-47c5-8a2f-2b77becc47bc(2 rows)

@marcospri marcospri marked this pull request as ready for review September 2, 2024 08:01
@marcospri marcospri changed the title WIP, store V13 assignment ID Store LTI1.3 assignment ID explicitly in the DB Sep 2, 2024
Base automatically changed from lti13-resource-id-model to main September 2, 2024 12:14
@marcospri marcospri merged commit 6f51b84 into main Sep 2, 2024
9 checks passed
@marcospri marcospri deleted the lti13-resource-id-code branch September 2, 2024 12:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants