fix: Various Edge-Case Bugs & Data Issues in modulestore_migrator - #37711

Merged

kdmccormick merged 29 commits into

masterfrom

kdmccormick/migration-api

Dec 18, 2025

Commits on Dec 18, 2025

fix(modulestore_migrator): No post-migration edits; No spurious sync; Better Python API
kdmccormick
committed
703e98d
fix: big simplifications to api
kdmccormick
committed
983b802
fix: migration
kdmccormick
committed
d4707df
fix: is_successfully_migrated
kdmccormick
committed
f043dd6
fix: rm test that I think doesn't make any sense
kdmccormick
committed
aaf6ff4
fix: better api method names
kdmccormick
committed
2178536
temp: don't fail fast
kdmccormick
committed
16dc4b8
fix: use correct api function
kdmccormick
committed
baea60b
fix: _slug vs. _key, and a forwarding test
kdmccormick
committed
ebdb452
refactor: rm pointless inversion of is_failed to is_successful
kdmccormick
committed
98fb819
revert: temp: fail fast tests
kdmccormick
committed
a5bd255
fix: handling of forward_source_to_target is None in tests
kdmccormick
committed
741d29f
fix: comment typo
kdmccormick
committed
c9c5500
fix: comment typo
kdmccormick
committed
69ed6d0
fix: remove unnecessary changes from readonly library effort
kdmccormick
committed
131be2a
fix: Non-admin should be able to migrate. Also, fix create_collection api docs
kdmccormick
committed
be51064
fix: rest api fixes & claude-written tests
kdmccormick
committed
e004f9b
style: simplify overly long multiline conditional

kdmccormick
and
ormsbee
committed
045e475
fix: dave & navin's review comments
kdmccormick
committed
c76357d
refactor: forward_$blah -> forwarding_for_$blah
kdmccormick
committed
627ae7d
docs: put back comment that i accientally removed
kdmccormick
committed
f660c78
fix: create_collection. also, strs -> enums in api
kdmccormick
committed
b9991dc
fix: typo'd name of has_studio_write_access
kdmccormick
committed
2803119
docs: improve two comments based on reviewer feedback
kdmccormick
committed
52485f9
fix: by default, URL slugs should not be preserved
kdmccormick
committed
eeaa9f1
docs: better forward_source_to_target default docs
kdmccormick
committed
e0e320b
fix: missing f-string in PermissionDenied

kdmccormick
and
ormsbee
committed
167595c
docs: more forwarding comments. also, rename forwarding read_api methods
kdmccormick
committed
64da92e
fix: extra period
kdmccormick
committed
88e5ffb

Navigation Menu

Uh oh!

fix: Various Edge-Case Bugs & Data Issues in modulestore_migrator - #37711

fix: Various Edge-Case Bugs & Data Issues in modulestore_migrator#37711
kdmccormick merged 29 commits into
masterfrom
kdmccormick/migration-api

Commits on Dec 18, 2025

fix(modulestore_migrator): No post-migration edits; No spurious sync; Better Python API

fix: big simplifications to api

fix: migration

fix: is_successfully_migrated

fix: rm test that I think doesn't make any sense

fix: better api method names

temp: don't fail fast

fix: use correct api function

fix: _slug vs. _key, and a forwarding test

refactor: rm pointless inversion of is_failed to is_successful

revert: temp: fail fast tests

fix: handling of forward_source_to_target is None in tests

fix: comment typo

fix: comment typo

fix: remove unnecessary changes from readonly library effort

fix: Non-admin should be able to migrate. Also, fix create_collection api docs

fix: rest api fixes & claude-written tests

style: simplify overly long multiline conditional

fix: dave & navin's review comments

refactor: forward_$blah -> forwarding_for_$blah

docs: put back comment that i accientally removed

fix: create_collection. also, strs -> enums in api

fix: typo'd name of has_studio_write_access

docs: improve two comments based on reviewer feedback

fix: by default, URL slugs should not be preserved

docs: better forward_source_to_target default docs

fix: missing f-string in PermissionDenied

docs: more forwarding comments. also, rename forwarding read_api methods

fix: extra period