Add mechanism for discovering tools in Canvas account, creating stub LtiTool records (#9) #287
Conversation
ssciolla
changed the title
TinyMCE Default Config Modification (tl-its-umich-edu#203) (tl-its-umich-edu#266)
| API_URL = options['api_url'] | ||
| canvas = Canvas(API_URL, API_KEY) | ||
|
|
||
| primary_acount = canvas.get_account(1) |
There was a problem hiding this comment.
Should probably make this account configurable. It want to be run on subaccounts.
| # avoid duplicates: get_or_create | ||
| if not LtiTool.objects.filter(canvas_id=ex_tool.id).exists(): | ||
| tool = LtiTool.objects.create( | ||
| name=ex_tool.name, |
There was a problem hiding this comment.
I think we'll want to append something to the name like "(PLACEHOLDER)".
We're also using the fact that the ID not being set means the tool is "disabled" now, we probably will need to add a new column to indicate this if we're having a new process to run this.
There was a problem hiding this comment.
@roujiawang added an is_public field now, so I think we're good on the second front. It might be useful to include placeholder text, not completely sold though.
| course_nav_placement = CanvasPlacement.objects.get(name='Course Navigation') | ||
| for ex_tool in ex_tool_list: | ||
| # for now, only care about tools with placement "course_navigation" | ||
| if ex_tool.__dict__["course_navigation"] is not None \ |
There was a problem hiding this comment.
We'll probably also want to make this configurable but this is fine hardcoding for now.
|
Good start on this feature idea! I'm sure we'll have a few more things to talk about with this and to get through the other ToDo items. |
…-explorer into issue-9-pull-list-of-canvas-available-tools
…/github.com/roujiawang/canvas-app-explorer into issue-9-pull-list-of-canvas-available-tools
#9
To Do
models, addpublicBooleanFieldfield for LTI Tool instances, and set this field toFalseby default.admin, display this field, and allow quick actions to change the value of this field for multiple instances.views, add a filter so that user view only lists tools whosepublicfield is set totrue.Trueor set toFalse, and the tool will display/hide accordingly in user viewCanvas idto display; that is, some tool with an invalidCanvas idwill not display to users even if this field is set as 'True'canvasapicode intoCanvasLtiManageror another adapter class