Skip to content
Open
Show file tree
Hide file tree
Changes from 3 commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions .importlinter
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@
root_packages =
openedx_learning
openedx_content
openedx_catalog
openedx_tagging
openedx_django_lib
openedx_core
Expand All @@ -23,8 +24,7 @@ layers =
# particular, openedx_tagging must never know that CBE exists.
openedx_learning

# Content: authoring-side models and APIs.
openedx_content
# Content (authoring-side models and APIs) and Catalog (CatalogCourse/CourseRun) as siblings
Comment thread
jesperhodge marked this conversation as resolved.

# Tagging is very simple & fundamental. Should probably not depend on any other Django apps.
openedx_tagging
Expand Down
3 changes: 3 additions & 0 deletions mypy.ini
Original file line number Diff line number Diff line change
Expand Up @@ -12,5 +12,8 @@ files =
[mypy-organizations.*]
follow_untyped_imports = True

[mypy-simple_history.*]
follow_untyped_imports = True

[mypy.plugins.django-stubs]
django_settings_module = "projects.dev"
4 changes: 4 additions & 0 deletions projects/dev.py
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,10 @@
# Open edX Organizations (dependency for openedx_catalog)
"organizations",

# Required for django-simple-history's admin integration and management commands;
# HistoricalRecords() alone does not need it.
"simple_history",

# Our Apps
"openedx_catalog",
"openedx_learning",
Expand Down
2 changes: 2 additions & 0 deletions requirements/base.in
Original file line number Diff line number Diff line change
Expand Up @@ -17,3 +17,5 @@ rules<4.0 # Django extension for rules-based authorization check
tomlkit # Parses and writes TOML configuration files

edx-organizations # Implemented the "Organization" model that CatalogCourse/CourseRun are keyed to

django-simple-history # History tracking for CBE criteria definitions, per ADR-0003
4 changes: 3 additions & 1 deletion requirements/base.txt
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,9 @@ django-crum==0.7.9
django-model-utils==5.0.0
# via edx-organizations
django-simple-history==3.13.0
# via edx-organizations
# via
# -r requirements/base.in
# edx-organizations
django-waffle==5.0.0
# via
# edx-django-utils
Expand Down
3 changes: 3 additions & 0 deletions test_settings.py
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,9 @@ def root(*args):
"organizations",
# django-rules based authorization
'rules.apps.AutodiscoverRulesConfig',
# Required for django-simple-history's admin integration and management commands;
# HistoricalRecords() alone does not need it.
"simple_history",
# Our own apps
"openedx_tagging",
"openedx_content",
Expand Down