Skip to content

Resource Links Create/Edit/Delete Trigger Live Event#40

Open
braysch wants to merge 3 commits intomasterfrom
bs/resource-links-live-event
Open

Resource Links Create/Edit/Delete Trigger Live Event#40
braysch wants to merge 3 commits intomasterfrom
bs/resource-links-live-event

Conversation

@braysch
Copy link
Copy Markdown

@braysch braysch commented Mar 17, 2026

There were no live events for Lti::ResourceLink. This adds lti_resource_link_created and lti_resource_link_updated events so downstream consumers can react when Canvas encounters a new LTI link in a course.

  • Added lti_resource_link_created and lti_resource_link_updated event methods to Canvas::LiveEvents with a shared
    payload helper
  • Registered Lti::ResourceLink in LiveEventsObserver and wired after_create/after_update in
    Canvas::LiveEventsCallbacks
  • Soft delete (.destroy) fires lti_resource_link_updated with workflow_state: "deleted" — no separate delete event

Payload includes: resource_link_id, resource_link_uuid, lookup_uuid, context_id, context_type,
context_external_tool_id, url, title, workflow_state

Verified with: unit tests, observer integration tests, and manual testing via fake Kinesis server

Bradan Schwanke and others added 3 commits March 17, 2026 10:14
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@braysch braysch requested a review from blunckr-aj March 17, 2026 19:05
Copy link
Copy Markdown

@JoshBothell JoshBothell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ryker has indicated that we are good to go ahead with these PRs to Instructure. In his words "you’re more familiar with the code at this point anyways."

This looks pretty much exactly the same as the new events I added for quizzes w/o assignment. I think we are good to send it over to Instructure. Justin also said that they should get auto assigned and we don't need to specifically notify him when we open a PR to Instructure.

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