panda_pal 5.16.0 → 5.16.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c78eb783dcf56cb075ce23d1218500128e6ac1ac58c3049bcfa13f6409835554
-  data.tar.gz: bb555a130fdab3e1805013700e04bddfe24a6c3460750ac2201a06f9dfe96c1d
+  metadata.gz: 2b33c2970defb1a7405a8482186720ab63a05b10db522e4c39ce56b9aa38c1b9
+  data.tar.gz: 6a44dc5db6d135882e4c2aefe8b7872ea87ccd066fe329b0764e2f8564694800
 SHA512:
-  metadata.gz: 5378ea91ce63e263dc93368f4d87cec35a14273b74f37452cec2b1dea67ff8bbb3a261e609bbc550bbcc4f08dac8c1de09e387e6ec8691e3d4901ac7b2af1e0a
-  data.tar.gz: aef30aa2d98b20f502a9274ce31c5bf17e66111c8a628cd61ce2765c06ef7fe280af459069a3fafe0a13aa885e9ba915f048553f14714e3ad274537b7bd4cfe3
+  metadata.gz: 589b2581d892bca49ba635d01a70b9db1f6044b9c885758fdbfb6d483ca3c91861bc22515fdbde2086499c7d380b96f199f8dd26e44e1cd900f2a3c3c72fc53a
+  data.tar.gz: 330d2c17013313a47f361f12709acd99291fe918cdea00405b7f76d55925557000c66b0eee08ea68049ece4acd77e86db1fd3acec73ccacaa3165e235cadcafa

data/README.md

@@ -166,7 +166,7 @@ PandaPal.stage_placement(:content_selection, {
 })
 ```
 
-2. **Include helpers** in your controller and build content items:
+2. **Include helpers** in your controller and build content items with **required `launch_type`**:
 ```ruby
 class ContentSelectionController < ApplicationController
   include PandaPal::DeepLinkingHelpers
@@ -174,8 +174,12 @@ class ContentSelectionController < ApplicationController
   def create
     content_items = [
       build_link_content_item(
-        url: launch_url_for_content(params[:content_id]),
-        title: 'Selected Content'
+        url: panda_pal.v1p3_resource_link_request_url,
+        title: 'Selected Content',
+        custom: {
+          content_id: params[:content_id],
+          launch_type: :content_launch  # REQUIRED for proper routing
+        }
       )
     ]
     render_deep_linking_response(content_items)
@@ -183,6 +187,14 @@ class ContentSelectionController < ApplicationController
 end
 ```
 
+3. **Add corresponding launch routes**:
+```ruby
+# config/routes.rb
+lti_nav content_launch: 'content#launch'
+```
+
+**⚠️ IMPORTANT**: All content items **MUST** include `launch_type` in the `custom` hash. This is used by the `resource_link_request` action to determine which route to redirect to when Canvas launches your content.
+
 **For complete configuration options, implementation examples, helper method documentation, and troubleshooting, see [DEEP_LINKING.md](DEEP_LINKING.md).**
 
 ## Implementating data segregation

data/app/controllers/panda_pal/lti_v1_p3_controller.rb

@@ -40,7 +40,7 @@ module PandaPal
     end
 
     def resource_link_request
-      ltype = @decoded_lti_jwt[LtiConstants::Canvas::PLACEMENT]
+      ltype = @decoded_lti_jwt[LtiConstants::Canvas::PLACEMENT] || @decoded_lti_jwt[LtiConstants::Claims::CUSTOM]&.dig("launch_type")
 
       if ltype
         current_session.data.merge!({

data/app/lib/panda_pal/deep_linking_helpers.rb

@@ -1,7 +1,26 @@
 module PandaPal
   module DeepLinkingHelpers
     # Build a link content item
-    def build_link_content_item(url:, title: nil, text: nil, icon: nil, thumbnail: nil, **options)
+    # @param url [String] The launch URL for the content
+    # @param title [String] The title of the content item
+    # @param text [String] Optional description text
+    # @param icon [String] Optional icon URL
+    # @param thumbnail [String] Optional thumbnail URL
+    # @param custom [Hash] Custom parameters - MUST include launch_type for proper routing
+    # @param options [Hash] Additional options to merge into the content item
+    #
+    # Example:
+    #   build_link_content_item(
+    #     url: panda_pal.v1p3_resource_link_request_url,
+    #     title: 'My Quiz',
+    #     custom: {
+    #       quiz_id: 123,
+    #       launch_type: :quiz_launch  # REQUIRED for proper routing
+    #     }
+    #   )
+    def build_link_content_item(url:, title: nil, text: nil, icon: nil, thumbnail: nil, custom: {}, **options)
+      validate_custom_launch_type!(custom)
+
       content_item = {
         type: 'link',
         url: url,
@@ -10,11 +29,21 @@ module PandaPal
       content_item[:text] = text if text.present?
       content_item[:icon] = icon if icon.present?
       content_item[:thumbnail] = thumbnail if thumbnail.present?
+      content_item[:custom] = custom if custom.present?
       content_item.merge(options)
     end
 
     # Build a file content item
-    def build_file_content_item(url:, title: nil, text: nil, media_type: nil, icon: nil, **options)
+    # @param url [String] The file URL
+    # @param title [String] The title of the content item
+    # @param text [String] Optional description text
+    # @param media_type [String] Optional media type
+    # @param icon [String] Optional icon URL
+    # @param custom [Hash] Custom parameters - MUST include launch_type for proper routing
+    # @param options [Hash] Additional options to merge into the content item
+    def build_file_content_item(url:, title: nil, text: nil, media_type: nil, icon: nil, custom: {}, **options)
+      validate_custom_launch_type!(custom)
+
       content_item = {
         type: 'file',
         url: url,
@@ -23,17 +52,26 @@ module PandaPal
       content_item[:text] = text if text.present?
       content_item[:mediaType] = media_type if media_type.present?
       content_item[:icon] = icon if icon.present?
+      content_item[:custom] = custom if custom.present?
       content_item.merge(options)
     end
 
     # Build an HTML content item
-    def build_html_content_item(html:, title: nil, text: nil, **options)
+    # @param html [String] The HTML content
+    # @param title [String] The title of the content item
+    # @param text [String] Optional description text
+    # @param custom [Hash] Custom parameters - MUST include launch_type for proper routing
+    # @param options [Hash] Additional options to merge into the content item
+    def build_html_content_item(html:, title: nil, text: nil, custom: {}, **options)
+      validate_custom_launch_type!(custom)
+
       content_item = {
         type: 'html',
         html: html,
         title: title
       }
       content_item[:text] = text if text.present?
+      content_item[:custom] = custom if custom.present?
       content_item.merge(options)
     end
 
@@ -68,6 +106,12 @@ module PandaPal
 
     private
 
+    def validate_custom_launch_type!(custom)
+      unless custom.is_a?(Hash) && custom.key?(:launch_type)
+        raise ArgumentError, "custom parameter must include :launch_type for proper routing. Example: custom: { launch_type: :my_content_type }"
+      end
+    end
+
     def build_deep_link_jwt(content_items, custom_deep_link_data)
       # Get issuer (client_id) and audience from session
       launch_params = current_session[:launch_params]

data/app/lib/panda_pal/launch_url_helpers.rb

@@ -52,7 +52,7 @@ module PandaPal
     def self.launch_route(opts, launch_type: nil)
       if opts.is_a?(Symbol) || opts.is_a?(String)
         launch_type = opts.to_sym
-        opts = PandaPal.lti_paths[launch_type]
+        opts = PandaPal.lti_paths[launch_type] || {}
       end
 
       if opts[:route_helper_key]

data/lib/panda_pal/version.rb

@@ -1,3 +1,3 @@
 module PandaPal
-  VERSION = "5.16.0"
+  VERSION = "5.16.1"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: panda_pal
 version: !ruby/object:Gem::Version
-  version: 5.16.0
+  version: 5.16.1
 platform: ruby
 authors:
 - Instructure CustomDev