Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix default of increase max CCX enrollments #30

Merged
merged 4 commits into from
Jun 28, 2023
Merged
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
6 changes: 3 additions & 3 deletions nau_openedx_extensions/message_gateway/tasks.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@

import logging

from celery import task # lint-amnesty, pylint: disable=import-error
from celery import shared_task # lint-amnesty, pylint: disable=import-error
from django.conf import settings
from django.contrib.auth.models import User # lint-amnesty, pylint: disable=imported-auth-user
from opaque_keys.edx.keys import CourseKey
Expand All @@ -16,7 +16,7 @@
log = logging.getLogger(__name__)


@task
@shared_task
def submit_bulk_course_message(message_id, course_id):
"""
Submit course messages. This will create the subtasks that submits
Expand Down Expand Up @@ -83,7 +83,7 @@ def _iterate_over_recipients(combined_set, batch_size):
num_items_queued += len(recipients)


@task
@shared_task
def submit_course_message(message_id, recipients):
"""
Send course message to the recipients given.
Expand Down
6 changes: 6 additions & 0 deletions nau_openedx_extensions/settings/production.py
Original file line number Diff line number Diff line change
Expand Up @@ -84,3 +84,9 @@ def plugin_settings(settings):
settings.XBLOCK_SETTINGS["ScormXBlock"] = {
"STORAGE_FUNC": scorm_xblock_storage,
}

#### Custom Courses for EDX (CCX) configuration
# Allow to increase the maximum number of studends allowed in a CCX (Custom Courses for edX).
settings.CCX_MAX_STUDENTS_ALLOWED = getattr(settings, "ENV_TOKENS", {}).get(
"CCX_MAX_STUDENTS_ALLOWED", 200
)
2 changes: 1 addition & 1 deletion requirements/base.in
Original file line number Diff line number Diff line change
Expand Up @@ -4,4 +4,4 @@ edx-opaque-keys[django]
six
future; python_version < "3.0"
web-fragments
openedx-filters==0.4.3
openedx-filters==0.7.0
2 changes: 1 addition & 1 deletion requirements/base.txt
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ celery==4.4.7 # via -c requirements/constraints.txt, -r requirements
django==2.2.25 # via -c requirements/constraints.txt, edx-opaque-keys, openedx-filters
edx-opaque-keys[django]==2.2.0 # via -c requirements/constraints.txt, -r requirements/base.in
kombu==4.6.11 # via celery
openedx-filters==0.4.3 # via -c requirements/constraints.txt, -r requirements/base.in
openedx-filters==0.7.0 # via -c requirements/constraints.txt, -r requirements/base.in
pbr==5.10.0 # via stevedore
pymongo==4.2.0 # via edx-opaque-keys
pytz==2022.2.1 # via celery, django
Expand Down
2 changes: 1 addition & 1 deletion requirements/constraints.txt
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,6 @@
celery<5.0
Django==2.2.25
edx-opaque-keys[django]==2.2.0
openedx-filters==0.4.3
openedx-filters==0.7.0
pip-tools<5.4
click==7.1.2
2 changes: 1 addition & 1 deletion requirements/test.txt
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ kombu==4.6.11 # via -r requirements/base.txt, celery
lazy-object-proxy==1.7.1 # via astroid
markupsafe==2.1.1 # via jinja2
mccabe==0.7.0 # via pylint
openedx-filters==0.4.3 # via -c requirements/constraints.txt, -r requirements/base.txt
openedx-filters==0.7.0 # via -c requirements/constraints.txt, -r requirements/base.txt
packaging==21.3 # via pytest
pbr==5.10.0 # via -r requirements/base.txt, stevedore
platformdirs==2.5.2 # via pylint
Expand Down