openequella / openEQUELLA

Core openEQUELLA sources
https://openequella.github.io/
Apache License 2.0
42 stars 44 forks source link

Canvas integration selection session doesn't close when selections have been made #2600

Closed SammyIsConfused closed 3 years ago

SammyIsConfused commented 3 years ago

Describe the bug A clear and concise description of what the bug is. In Canvas integration, on Save the selection session doesn’t close, it stays open.

This only occurs if there are selections made - without selections it simply closes the selection sessions when clicking Save and redirects back to the Canvas course page. Since there is a tooltip “x resource(s) added to Canvas” it seems that it was designed this way - potentially this was not possible in the Canvas integration? I have attached clips of this happening in the old and the new UI. It happens on stable as well, so 2020.1 is affected ( and probably older too).

To Reproduce Steps to reproduce the behavior:

  1. In a Canvas set up with oEQ integration, visit a course and click "Select resources from openEQUELLA". Wait for the selection session to open.
  2. Once in the session, make some selections and click Save.
  3. Note that the selection session stays open, with a message confirming that the selections have been added to Canvas.
  4. Save again, this time with nothing selected.
  5. Note that the selection session closes and you are redirected to the Canvas course page.

Expected behavior Once selections have been saved, the selection session should close and the user should be redirected to the Canvas course page. (I.E it should work the same as with other LMS selection sessions).

Screenshots Peek 2020-12-14 16-22 Peek 2020-12-14 16-18

Stacktrace No stacktrace for this issue, it fails to close silently (potentially because it was designed to work like this)

Platform:

Additional context Add any other context about the problem here.

cathfitz commented 3 years ago

Designed this way