User can add MyLA to a course when they have Teacher role in at least one section

[jennlove-um]

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

When a user is enrolled in multiple sections of a course, if one of those enrollments is a teacher enrollment, then that user can add MyLA to the course.

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

When a user has a Teacher enrollment in one section and a TA enrollment in another section of a course, they are not able to add MyLA to the course. They can use the Navigation to add MyLA, but when they click on MyLA, they get the message: Course NNNNN has not been set up in MyLA. Contact your instructor, who can enable the visualizations by clicking on MyLA in the course navigation.

Steps to Reproduce :

1. Identify a course that has a user with enrollments in multiple sections with at least one Teacher enrollment.
   a. You can also add a new user to your course with +People to these two sections and different enrollments.
2. Add MyLA to the course from the Settings > Navigation
3. Click on MyLA

[jonespm]

It looks like this was explicitly changed to work like this in #1381 and was a test case to pass. There is a comment in the code.

# Checking if user only has Instructor role in a course to enable MyLA in courses.
# A TA could be an instructor in an course section so his role will be both TA and Instructor.
# we don't want TA to enable the MyLA data extraction step.

[jennlove-um]

You're right. I had forgotten that we made that decision. After having 3 instructors report that they were unable to add MyLA to their courses because they were both Teachers and TAs, I'm rethinking that choice.

…

On Tue, Jan 28, 2025 at 5:26 PM Code Hugger (Matthew Jones) < ***@***.***> wrote: It *looks* like this was explicitly changed to work like this in #1381 <#1381> and was a test case to pass. There is a comment in the code. # Checking if user only has Instructor role in a course to enable MyLA in courses. # A TA could be an instructor in an course section so his role will be both TA and Instructor. # we don't want TA to enable the MyLA data extraction step. — Reply to this email directly, view it on GitHub <#1631 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AJOML4D7E4WV57ZNKDESVLD2M77Y7AVCNFSM6AAAAABWBQAJ2CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMMRQGE4DCNJSG4> . You are receiving this because you authored the thread.Message ID: ***@***.***>

-- *Jennifer Love* (she, her, hers <http://www.mypronouns.org/>) <https://www.name-coach.com/jennifer-love-003e538b-03de-454b-a695-2692c7ce116e> *Business Systems Analyst, ITS Teaching & Learning* Information and Technology Services | University of Michigan Check out our Generative AI workshop series <https://its.umich.edu/teaching-learning/workshops/training> and teaching resources <https://academictechnology.umich.edu/instructional-resources/generative-ai>!

[jonespm]

It looks like it might actually have been done 5 years ago in #982. I'm not sure if this was done for passing the certification or if we'd made this decision back then.

I'm not sure what issues a user having 2 roles in a course might cause. As long as it doesn't have any issues with the certification this is a pretty straightforward change if we were to do this. We might want to put it behind a setting incase some schools didn't want this.

We also wanted to rewrite LTI to use the Academic Innovation library as mentioned on #1304 but not sure when/if that would happen, but this seems like a smaller somewhat related to that.