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

Students receive the wrong error message if MyLA isn't connected to data in Canvas #1540

Closed
jennlove-um opened this issue Sep 19, 2023 · 4 comments · Fixed by #1561
Closed
Assignees
Labels
🪳 bug Something isn't working

Comments

@jennlove-um
Copy link
Contributor

Expected behavior (A description of what you expected to happen) :

When a Teacher has added MyLA to a course but has not clicked on the link to enable the course in MyLA, it is not completely set up for students to use. When a student clicks on the link before the teacher has clicked on LTI to enable the course in MyLA, they see the following message: This course has not been set up in MyLA. Contact your instructor, who can enable the visualizations by clicking on MyLA in the course navigation.

Describe the bug (Tell us what happens instead of the expected behavior) :

Instead, the error that displays in production and test environments is "Something went wrong; please try again later."

Steps to Reproduce :

  1. Add MyLA in the course navigation (do not click on the link once it is added)
  2. As a student, click on the MyLA link in the navigation
  3. The wrong error message displays
@jennlove-um jennlove-um added the 🪳 bug Something isn't working label Sep 19, 2023
@jennlove-um jennlove-um added this to To do in MyLA-Default-Project via automation Sep 19, 2023
@jennlove-um
Copy link
Contributor Author

The correct error message DOES display if the student tries to go to MyLA in a course where MyLA has not been added at all by changing the course ID in the URL.

@jennlove-um jennlove-um removed this from To do in MyLA-Default-Project Sep 27, 2023
@jennlove-um jennlove-um added this to To do in MyLA-2024.01.01 via automation Sep 27, 2023
@jennlove-um jennlove-um self-assigned this Oct 12, 2023
@jennlove-um
Copy link
Contributor Author

Add details about what message should display and evaluate priority.

@jennlove-um
Copy link
Contributor Author

The error message should be "This course has not been set up in MyLA. Contact your instructor, who can enable the visualizations by clicking on MyLA in the course navigation."

@zqian zqian self-assigned this Jan 16, 2024
@jennlove-um jennlove-um moved this from To do to In progress in MyLA-2024.01.01 Jan 17, 2024
zqian added a commit to zqian/my-learning-analytics that referenced this issue Jan 25, 2024
…ed when student invokes MyLA before fully setup
zqian added a commit that referenced this issue Jan 25, 2024
…invokes MyLA before fully setup (#1561)

* issue #1540 issue_1540 empty error message was returned when student invokes MyLA before fully setup

* Update Course.js with formatting changes

* Fix for data to return status if statusText is not available

---------

Co-authored-by: Code Hugger (Matthew Jones) <jonespm@umich.edu>
@zqian zqian moved this from In progress to Review/QA in MyLA-2024.01.01 Jan 25, 2024
@jennlove-um
Copy link
Contributor Author

Testing passes in beta

@jennlove-um jennlove-um moved this from Review/QA to Done in MyLA-2024.01.01 Jan 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🪳 bug Something isn't working
Projects
No open projects
3 participants