Page Comparison

...

1. django-push-notificationsis not supported currently for Python 3.10 and Django 4? Do we want to contribute this package or to use Firebase admin SDK.

   1. Findings: The current package isn’t fully compatible wuthwith python3.10 and higher versions of python because of the APNS dependency used, there’s an active work is in progress on the PR (last reply 04/08/2024) https://github.com/jazzband/django-push-notifications/pull/696, so we believe this issue could be solved really soon. And there’re no big risks for using django-push-notifications.

2. Where the push notifications app should be initialized? Example: we specify FCM_API_KEY and initialization should be in the edx-ace or in the edx-platform.

   1. Solution: Approach the same way as Emails configuration, use Django settings in ace_common https://github.com/openedx/edx-platform/blob/ddb407a3f2c75bb65746a4c16ba32b2c26296019/openedx/core/djangoapps/ace_common/settings/common.py edx-platform to init variables, and ENV Tokens approach to override it for production installations. Use those settings in edx-ace code to configure a firebase client.

3. In case we use django-push-notifications, what is the best way to extend INSTALLED_APPS? Install it straight into edx-platform or only edx-ace?

   1. Patch Installed_apps in a plugin settings, if Push Notifications are activated - https://github.com/openedx/edx-platform/blob/ddb407a3f2c75bb65746a4c16ba32b2c26296019/openedx/core/djangoapps/ace_common/settings/common.py

4. If we don’t use django-push-notifications can we create models and endpoint in the edx-ace

   1. We will use it, also some specific models and Views can be exposed via mobile_api application in LMS

New questions & updates (2024.04.09):

...