Skip to content

build: Upgrade to openedx-core v1.0#38435

Open
kdmccormick wants to merge 10 commits intomasterfrom
kdmccormick/core-1
Open

build: Upgrade to openedx-core v1.0#38435
kdmccormick wants to merge 10 commits intomasterfrom
kdmccormick/core-1

Conversation

@kdmccormick
Copy link
Copy Markdown
Member

@kdmccormick kdmccormick commented Apr 23, 2026

kdmccormick and others added 9 commits April 23, 2026 16:38
Renames the openedx_django_lib.fields import in EntityLinkBase from
the removed key_field helper to ref_field.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…0.43.0

Updates callers of get_learning_package_by_key (renamed to
get_learning_package_by_ref), create_learning_package, and
update_learning_package to use the new package_ref kwarg, and
switches .key attribute reads to .package_ref.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
… 0.43.0

Switches callers of get_publishable_entity_by_key (now _by_ref) to the
new name, and renames .key attribute reads on PublishableEntity,
Component, and Container (via PublishableEntityMixin) to .entity_ref.
Query filters using key__in/entity__key become entity_ref__in/
entity__entity_ref. The set_library_item_collections param entity_key
is renamed to entity_ref for consistency.

Collection.key reads are intentionally left for the collection_code
rename in a later commit.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…re 0.43.0

Updates callers of get_component_by_key/component_exists_by_key (now
_by_code) and switches the local_key kwarg on create_component,
create_component_and_version, and related queries to component_code.
Also renames component.local_key reads to component.component_code and
adjusts a modulestore_migrator container query that filtered on
publishable_entity__key (now entity_ref).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…edx-core 0.43.0

Switches get_container_by_key to get_container_by_code and renames the
container creation kwarg from key to container_code for both
create_container_and_version and its platform wrapper. Updates
ComponentVersionMedia accesses: the model field .key becomes .path,
and create_component_version_media's key kwarg becomes path.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…-core 0.43.0

Renames all Collection APIs that took key/collection_key to use
collection_code: create_collection, update_collection, delete_collection,
restore_collection, add_to_collection, etc. Switches Collection.key
attribute reads to .collection_code across tests, signal handlers,
search indexers, and modulestore_migrator. Filters like
target_collection__key become target_collection__collection_code.

Also updates the library restore serializer to track the renamed
lp_restored_data fields: archive_org_key -> archive_org_code,
archive_slug -> archive_package_code, key -> package_ref,
archive_lp_key -> archive_package_ref. The archive_org_code and
archive_package_code fields now allow None, since openedx-core no
longer raises ValueError when the archive_package_ref cannot be
parsed as {prefix}:{org_code}:{package_code}.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
	fix: build locator with container_code

	fix: pylint and mypy

	fix: queries for search index

	fix: some missed cvm.key -> cvm.path

	fix: undo breaking library changes

	fix: openedx-core no longer raises integrityerror on conflict

	fix: misses in modulestore_migrator

	fix: search tests

	docs: Improve collection_code/key TODO comment
This is necessary because we are no longer presuming that package_ref
follows the same format as a Content Library. In the future, we may
want a more graceful way of handling this.
@kdmccormick kdmccormick changed the title build: Upgrade to openedx-core V1 build: Upgrade to openedx-core v1.0 Apr 23, 2026
@kdmccormick kdmccormick marked this pull request as ready for review April 23, 2026 20:55
Base automatically changed from kdmccormick/keys to master April 23, 2026 21:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant