Skip to content

Clarify Resource Link ID terminology#287

Draft
Fdavidtr wants to merge 2 commits into
Cvmcosta:masterfrom
Fdavidtr:clarify-resource-link-id-v2
Draft

Clarify Resource Link ID terminology#287
Fdavidtr wants to merge 2 commits into
Cvmcosta:masterfrom
Fdavidtr:clarify-resource-link-id-v2

Conversation

@Fdavidtr

Copy link
Copy Markdown

What changed

  • Clarified internal provider naming so code and generated dist use resourceLinkId when reading the LTI resource_link.id claim.
  • Expanded grading docs to explain the difference between AGS resourceLinkId and resourceId, including the boolean filter vs string payload usage.
  • Clarified Names and Roles docs to state that resourceLinkId: true sends the current Resource Link ID as the rlid parameter.

Why

The existing terminology made it easy to confuse the LTI Resource Link identifier with AGS line item resourceId, which creates friction for users reading the docs and code.

Impact

  • No runtime behavior change intended.
  • Documentation should make intent and API usage clearer.

@Fdavidtr Fdavidtr marked this pull request as ready for review April 25, 2026 14:29
@Fdavidtr Fdavidtr marked this pull request as draft April 25, 2026 20:47
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.

1 participant