Skip to content
Open
Show file tree
Hide file tree
Changes from all 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
5 changes: 3 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,8 @@ 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.
openedx_content | openedx_catalog

# 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