Skip to content

Commit

Permalink
feat: replace signal handler with openedx_event producer config
Browse files Browse the repository at this point in the history
  • Loading branch information
navinkarkera committed Aug 29, 2023
1 parent 512c08d commit 9a1b6ec
Show file tree
Hide file tree
Showing 11 changed files with 232 additions and 260 deletions.
10 changes: 7 additions & 3 deletions README.rst
Original file line number Diff line number Diff line change
Expand Up @@ -105,9 +105,13 @@ settings: ``lms/envs/common.py``
# helps to configure probability of displaying the verification forms. Values in range 0 to 1 are allowed, where 0
# means never and 1 means always display. Default value is 0.5 i.e. 50% chance of displaying the form.
SHOW_SKILL_VERIFICATION_PROBABILITY = 0.5
# Optionally update topic name for verification event emitted when a user verifies tags for an xblock.
EVENT_BUS_XBLOCK_VERIFICATION_TOPIC = "learning-custom-xblock-skill-verfied"
# Include the XBLOCK_SKILL_VERIFIED signal in `EVENT_BUS_PRODUCER_CONFIG` to push the event to event bus
# Note: Include `openedx_events` in `INSTALLED_APPS` setting in the host application
EVENT_BUS_PRODUCER_CONFIG = {
'org.openedx.learning.xblock.skill.verified.v1': [
{'topic': 'learning-custom-xblock-skill-verfied', 'event_key_field': 'xblock_info.usage_key', 'enabled': True},
],
}
Developing
==========
Expand Down
16 changes: 9 additions & 7 deletions requirements/base.txt
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ asgiref==3.7.2
# via django
attrs==23.1.0
# via openedx-events
django==3.2.19
django==3.2.20
# via
# -c https://raw.githubusercontent.com/edx/edx-lint/master/edx_lint/files/common_constraints.txt
# -r requirements/base.in
Expand All @@ -17,13 +17,13 @@ django==3.2.19
# openedx-filters
django-model-utils==4.3.1
# via -r requirements/base.in
edx-opaque-keys[django]==2.3.0
edx-opaque-keys[django]==2.5.0
# via openedx-events
fastavro==1.7.4
fastavro==1.8.2
# via openedx-events
openedx-events==8.0.1
openedx-events==8.6.0
# via -r requirements/base.in
openedx-filters==1.3.0
openedx-filters==1.6.0
# via -r requirements/base.in
pbr==5.11.1
# via stevedore
Expand All @@ -35,5 +35,7 @@ sqlparse==0.4.4
# via django
stevedore==5.1.0
# via edx-opaque-keys
typing-extensions==4.6.3
# via asgiref
typing-extensions==4.7.1
# via
# asgiref
# edx-opaque-keys
14 changes: 8 additions & 6 deletions requirements/ci.txt
Original file line number Diff line number Diff line change
Expand Up @@ -4,17 +4,17 @@
#
# make upgrade
#
distlib==0.3.6
distlib==0.3.7
# via virtualenv
filelock==3.12.2
filelock==3.12.3
# via
# tox
# virtualenv
packaging==23.1
# via tox
platformdirs==3.5.3
platformdirs==3.10.0
# via virtualenv
pluggy==1.0.0
pluggy==1.3.0
# via tox
py==1.11.0
# via tox
Expand All @@ -27,7 +27,9 @@ tox==3.28.0
# -c https://raw.githubusercontent.com/edx/edx-lint/master/edx_lint/files/common_constraints.txt
# -r requirements/ci.in
# tox-battery
tox-battery==0.6.1
tox-battery==0.6.2
# via -r requirements/ci.in
virtualenv==20.23.0
typing-extensions==4.7.1
# via filelock
virtualenv==20.24.3
# via tox
Loading

0 comments on commit 9a1b6ec

Please sign in to comment.