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.

Sign up for GitHub

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

Jump to bottom

Incorrect privacy policy infomation is shown in LTI exercises #1342

Open
ihalaij1 opened this issue Feb 20, 2024 · 1 comment
Open

Incorrect privacy policy infomation is shown in LTI exercises #1342

ihalaij1 opened this issue Feb 20, 2024 · 1 comment
Labels
area: LTI LTI protocol (Learning Tools Interoperability, worldwide standard) effort: days Estimated to take less than one week, from the creation of a new branch to the merging experience: beginner required knowledge estimate requires: priority Currently using this label to flag issues that need EDIT decision ASAP (even if there was priority) type: bug This is a bug

Comments

@ihalaij1
Copy link
Contributor

ihalaij1 commented Feb 20, 2024 •
edited
Loading

image

https://plus.cs.aalto.fi/tietotekniikka-sovelluksissa/2024/simulink1/simulink-uusi/#chapter-exercise-1
@ihalaij1 ihalaij1 moved this to Todo in A+ sprints Feb 20, 2024
@murhum1 murhum1 added type: bug This is a bug area: LTI LTI protocol (Learning Tools Interoperability, worldwide standard) effort: days Estimated to take less than one week, from the creation of a new branch to the merging experience: beginner required knowledge estimate requires: priority Currently using this label to flag issues that need EDIT decision ASAP (even if there was priority) labels Apr 17, 2024
@murhum1
Copy link
Contributor

murhum1 commented Apr 17, 2024

What's the specific issue here? Is it the translation of the second part itself, or is it that the second part shouldn't be included at all since no personal information is being sent?

  • Looking at the code for the LTI1p3Service class, it looks like the user's A+ id will be shared with the tool in any case. Unsure if that counts as personal data?
  • Semi-related, in any case, I believe we should define a sends_user_info method for the LTI1p3Service class, that would just check the value of share_name, or just True if the user id counts as personal data.

Maybe I'll consult Anni about these.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area: LTI LTI protocol (Learning Tools Interoperability, worldwide standard) effort: days Estimated to take less than one week, from the creation of a new branch to the merging experience: beginner required knowledge estimate requires: priority Currently using this label to flag issues that need EDIT decision ASAP (even if there was priority) type: bug This is a bug
Projects
A+ sprints
Status: Todo
Milestone
No milestone
Development

No branches or pull requests
2 participants
@murhum1 @ihalaij1