Uploaded image for project: 'Sakai'
  1. Sakai
  2. SAK-44876

Lessons: Instructor using add External Tool should not have the Use LTI Advantage Auto Configuration button

    XMLWordPrintable

    Details

    • 21 Status:
      Verified
    • Test Plan:
      Hide
      1. Have the browser console open
      2. As the instructor, click on Lessons
      3. Click Add Content > Add External Tool
      4. Click Manage or Install Tools
      5. Click Install 1.x Tool in the pop-up

      Expected: Use LTI Advantage Auto Configuration button should not display to the instructor

      Show
      Have the browser console open As the instructor, click on Lessons Click Add Content > Add External Tool Click Manage or Install Tools Click Install 1.x Tool in the pop-up Expected: Use LTI Advantage Auto Configuration button should not display to the instructor

      Description

      Since the new LTI Advantage auto-config option is expected to be merged to 21, I'm setting please merge for 21x. This is related to SAK-44055.

      If the instructor tries to add an external tool through Lessons, once they get into the screen to actually add the tool info, the "Use LTI Advantage Auto Configuration" button displays. I think since the instructor is not able to add an LTI Advantage tool, that button should not display for the instructor.

      If the instructor clicks the button, they get kicked out of that window and there is a console error.

      sakai.basiclti.admin.helper.helper?panel=ToolInsert:48 Uncaught ReferenceError: showIframe is not defined
      at HTMLAnchorElement.onclick (sakai.basiclti.admin.helper.helper?panel=ToolInsert:48)
      onclick @ sakai.basiclti.admin.helper.helper?panel=ToolInsert:48

       

        Gliffy Diagrams

          Zeplin

            Attachments

              Issue Links

                Activity

                  People

                  Assignee:
                  csev Charles Severance
                  Reporter:
                  agschmidt Andrea Schmidt
                  Votes:
                  0 Vote for this issue
                  Watchers:
                  2 Start watching this issue

                    Dates

                    Created:
                    Updated:
                    Resolved:

                      Git Integration