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

Change LTI session verification methods for API requests in task view #373

Open
AndresFelipeCastellanos opened this issue Oct 20, 2021 · 0 comments
Labels
Change request New feature or request Low Priority Issues with the lowest priority, the fix of this is not very urgent. Task Issue related to a task
Projects

Comments

@AndresFelipeCastellanos
Copy link
Collaborator

Is your feature request related to a problem? Please describe.
When setting up a course to use LTI, all API requests in tasks from plugins (code preview, user hints), services (Linter, Python Tutor, Analytics), and send submissions, needs to verify that user session is LTI. However, each API request has its own method to check if is LTI session individually. So when a new API class is added, all user session verification methods must be created for each request.

Describe the solution you'd like
We need to add a different method to check if is a LTI task for all the API requests in task view, instead of creating a new method for each request.

@AndresFelipeCastellanos AndresFelipeCastellanos added Change request New feature or request Feature request New feature request Medium Priority Issues with middle priority. The fix may not be urgent but necessary for production. Task Issue related to a task and removed Feature request New feature request labels Oct 20, 2021
@AndresFelipeCastellanos AndresFelipeCastellanos added this to Not Yet in UNCode via automation Oct 20, 2021
@AndresFelipeCastellanos AndresFelipeCastellanos moved this from Not Yet to To Do in UNCode Oct 20, 2021
@ferestrepoca ferestrepoca moved this from To Do to Not Yet in UNCode May 7, 2022
@ferestrepoca ferestrepoca added Low Priority Issues with the lowest priority, the fix of this is not very urgent. and removed Medium Priority Issues with middle priority. The fix may not be urgent but necessary for production. labels May 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Change request New feature or request Low Priority Issues with the lowest priority, the fix of this is not very urgent. Task Issue related to a task
Projects
UNCode
  
Not Yet
Development

No branches or pull requests

2 participants