← Back to team overview

mahara-contributors team mailing list archive

[Bug 1948744] [NEW] LTI Advantage: Remove deep link required field

 

Public bug reported:

This affects Mahara 21.10 only

When you set up the LTI Advantage integration, there are three fields for deployment IDs:
- Basic launch
- NRPS
- Deep link (portfolio list).


Currently, either Basic launch or NRPS AND Deep link are required and can't take the same deployment IDs. Sometimes we only get one deployment ID though from the LMS. Since LTI doesn't really care which deployment ID is listed where (it's a convenience on the Mahara site to distinguish them if there are multiple deployment IDs, I suggest:

- Remove the requirement for 'Deep link' to be filled in.
- Give it the # so that either Basic launch, NRPS, or Deep Link need to be filled in.
- Rename the first deployment ID option 'Basic Launch' to 'Main deployment ID'. Add the description: If your platform only has one deployment ID, enter it here.
- Rename 'NRPS' to 'Additional deployment ID 1'. Description: Enter an additional deployment ID if needed, e.g. if you have separate deployments for NRPS or deep links.
- Rename 'Deep Link (portfolio list)' to 'Additional deployment ID 2'. Description: Enter an additional deployment ID if needed, e.g. if you have separate deployments for NRPS or deep links.

And while we are changing lang strings, can you please change the following for the drop-down menu on /webservice/admin/oauthv1sregister.php. The string ID can stay the same:
- 'LTI advantage integration' to 'LTI Advantage' 
- 'LTI integration' to 'LTI 1.1'

** Affects: mahara
     Importance: High
     Assignee: Gold (gold.catalyst)
         Status: Confirmed

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: mahara-contributors
https://bugs.launchpad.net/bugs/1948744

Title:
  LTI Advantage: Remove deep link required field

Status in Mahara:
  Confirmed

Bug description:
  This affects Mahara 21.10 only

  When you set up the LTI Advantage integration, there are three fields for deployment IDs:
  - Basic launch
  - NRPS
  - Deep link (portfolio list).

  
  Currently, either Basic launch or NRPS AND Deep link are required and can't take the same deployment IDs. Sometimes we only get one deployment ID though from the LMS. Since LTI doesn't really care which deployment ID is listed where (it's a convenience on the Mahara site to distinguish them if there are multiple deployment IDs, I suggest:

  - Remove the requirement for 'Deep link' to be filled in.
  - Give it the # so that either Basic launch, NRPS, or Deep Link need to be filled in.
  - Rename the first deployment ID option 'Basic Launch' to 'Main deployment ID'. Add the description: If your platform only has one deployment ID, enter it here.
  - Rename 'NRPS' to 'Additional deployment ID 1'. Description: Enter an additional deployment ID if needed, e.g. if you have separate deployments for NRPS or deep links.
  - Rename 'Deep Link (portfolio list)' to 'Additional deployment ID 2'. Description: Enter an additional deployment ID if needed, e.g. if you have separate deployments for NRPS or deep links.

  And while we are changing lang strings, can you please change the following for the drop-down menu on /webservice/admin/oauthv1sregister.php. The string ID can stay the same:
  - 'LTI advantage integration' to 'LTI Advantage' 
  - 'LTI integration' to 'LTI 1.1'

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1948744/+subscriptions



Follow ups