Closed jonespm closed 2 years ago
So, probably use a data migration to add the available Canvas Placements.
I believe the idea is we'd sync the placements for each tool from Canvas via API eventually. This is not going to happen in the short term, so we're hardcoding them for now. I think we have it as a table so it can be dynamic and not require code changes if these placements ever change.
@jonespm, how about something like this? https://github.com/ssciolla/canvas-app-explorer/commit/d0aa6cc2fbf7112b5b02228db9767e9c89342113
If it looks okay, I will open a PR after #134 is merged.
It would be nice if we pre-populated the stables like CanvasPlacement to have a few of the known defined placements. We also should pre-populate any other tables.