Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Blackboard selection session is broken in new UI #1725

Closed
cbeach47 opened this issue May 12, 2020 · 3 comments
Closed

Blackboard selection session is broken in new UI #1725

cbeach47 opened this issue May 12, 2020 · 3 comments

Comments

@cbeach47
Copy link
Contributor

Describe the bug
The final call to ltireturn.do (generally Blackboard) in a selection session loses it's parameters, thus keeping the user in oEQ and not creating the LTI link in the calling LTI consumer (Blackboard).

To Reproduce
Steps to reproduce the behavior:

  1. Setting an LTI Tool Provider for oEQ in Bb
  2. In Blackboard, access the 'Build content' > oEQ LTI launch
  3. Select a piece of content
  4. Return the selection
  5. Note how you get a relatively blank oEQ page and the link isn't created in Bb

Expected behavior
The link should be created in Bb and either the selection session should disappear or a notification appear that the link was created

Screenshots
image

Platform:

  • OpenEquella Version: 2020.1.0
  • Browser: Confirmed in Chrome and Firefox

Additional context
Switching to the old UI allows the Bb selection session to work.

No errors are thrown in the logs

@edalex-ian
Copy link
Member

Heya @cbeach47

As per @SammyIsConfused 's comment over in the PR for Hotfix 2020.1.2 (#1738), we possibly have a fix in this which we needed to do for Canvas (#1744). If you're able to test against Bb that'd be great. 👍

@cbeach47
Copy link
Contributor Author

Cool, I'll take a look Tuesday / Wednesday of next week

@cbeach47
Copy link
Contributor Author

Once the PR for Hotfix 2020.1.2 ( #1738 ) is complete and merged up into develop, this should be resolved. Testing notes in the #1738 comments. Leaving in 'Ready for Testing' until then.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants