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

Create a plugin for Canvas integration #3

Closed
5 tasks done
pdpinch opened this issue Oct 6, 2021 · 4 comments
Closed
5 tasks done

Create a plugin for Canvas integration #3

pdpinch opened this issue Oct 6, 2021 · 4 comments
Assignees
Labels

Comments

@pdpinch
Copy link
Member

pdpinch commented Oct 6, 2021

In the lilac branch for residential, we added a great deal of code to allow for Canvas integration. We should put as much as possible of it into a django plugin that can be pip installed, and run within the edx-platform process.

acceptance criteria

@pdpinch pdpinch added the epic label Oct 6, 2021
@arslanashraf7 arslanashraf7 self-assigned this Oct 26, 2021
@arslanashraf7
Copy link
Contributor

@pdpinch Do we have any testing instructions/documentation on how to test Canvas integration? My plan is to actually test it before moving forward to identify the whole scope of the canvas integration.

@arslanashraf7
Copy link
Contributor

@pdpinch There are a few things regarding:

remove any Stellar/LMOD remote gradebook code

  1. Could you give a pointer about, When we talk about Stellar/LMOD part what is it that we're looking at? (Some third-party integration maybe)
  2. Or, Do we need to just remove the remote_gradebook Django app? If we go for removing this app we might need to port some of its functionality to canvas integration since canvas app uses its API (Ref:)
  3. If we don't want to remove the whole remote_gradebook app we might be looking for two things:
    • As we are creating canvas_integration pluggable app we might need to do the same for remote_gradebook and make that a pluggable app too.
    • OR should we merge the remote_gradebook and canvas integration into a single app (I expect the chance of more breaking changes)

@pdpinch
Copy link
Member Author

pdpinch commented Nov 2, 2021

Sorry this wasn't clear. I believe remote_gradebook is the app that connects LMS to Stellar/LMOD. We don't want to maintain that connection any more, so you should remove as much of it as you can.

I think it would be best to move any canvas dependencies into the canvas_integration app, but I'm not sure what's easiest.

@arslanashraf7
Copy link
Contributor

Closed via #8, mitodl/edx-platform#274 & as additionally with a documentation PR (#19).

Note: I have created some other tickets(#10, #11, #12, #13, #16) in extensions to this epic but they are mostly enhancements/refactor. so I'm closing this ticket and we can work on them independently when suitable.

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

No branches or pull requests

2 participants