ylqjgm
/
canvas-lms
mirror of https://github.com/instructure/canvas-lms.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity
canvas-lms/gems/plugins/academic_benchmark
History
Augusto Callejas 1e7ff0a8c4 Skip setting root account on AB imports 
fixes OUT-3710

flag=none

Content migrations now automatically set their
root account from their context. This caused an
issue on Academic Benchmarks imports because the
context was the Site Admin context, which does
not exist in the current shard's accounts table,
causing a foreign key check to fail. Instead,
we skip setting the root account id altogether.

test plan:
  - set up Academic Benchmarks importing
  - create a second shard
  - confirm that importing works in that
    second shard without failing

Change-Id: Ibf1e4c1e94cd788b6c50203bf93bb1d80779b847
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/239741
Reviewed-by: Michael Brewer-Davis <mbd@instructure.com>
Reviewed-by: Pat Renner <prenner@instructure.com>
Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com>
QA-Review: Pat Renner <prenner@instructure.com>
Product-Review: Augusto Callejas <acallejas@instructure.com>
 		2020-06-10 18:45:43 +00:00
..
app/views/academic_benchmark Remove Academic Benchmark v1 code 2020-01-22 23:50:34 +00:00
lib Skip setting root account on AB imports 2020-06-10 18:45:43 +00:00
spec_canvas Skip setting root account on AB imports 2020-06-10 18:45:43 +00:00
Gemfile gemify acacemic_benchmark 2014-09-15 15:10:27 +00:00
academic_benchmark.gemspec rails 5.2 2018-04-25 19:15:20 +00:00