openedx / edx-platform

The Open edX LMS & Studio, powering education sites around the world!
https://openedx.org
GNU Affero General Public License v3.0
7.36k stars 3.85k forks source link

feat: [FC-0047] add mobile push notifications functionality #34971

Closed NiedielnitsevIvan closed 2 weeks ago

NiedielnitsevIvan commented 3 months ago

Description

This PR addressed to adding the ability to send push notifications to the OeX mobile app on IOS and Android, using edx-ace functionality and Firebase Cloud Messaging as a cloud provider.

Some push notifications are also added here directly, namely:

Useful information to include:

Supporting information

FC-0047 TBD

Testing instructions

  1. Install edx-ace version 1.9.0.
  2. Add FIREBASE_CREDENTIALS or FIREBASE_CREDENTIALS_PATH to the platform settings.
  3. Add a new push_notification channel to ACE_ENABLED_CHANEL.
  4. Log in to OeX.
  5. Install the OeX mobile application on your device.
  6. Add your device token to FCM Devices with a link to your user from the admin panel.
  7. Enroll/unenroll the user(from step 7) in the “Instructor” tab to trigger the push notification event.
  8. Make sure you receive push notifications on your device.

Deadline

"None"

Other information

openedx-webhooks commented 3 months ago

Thanks for the pull request, @NiedielnitsevIvan!

What's next?

Please work through the following steps to get your changes ready for engineering review:

:radio_button: Get product approval

If you haven't already, check this list to see if your contribution needs to go through the product review process.

:radio_button: Provide context

To help your reviewers and other members of the community understand the purpose and larger context of your changes, feel free to add as much of the following information to the PR description as you can:

:radio_button: Get a green build

If one or more checks are failing, continue working on your changes until this is no longer the case and your build turns green.

:radio_button: Let us know that your PR is ready for review:

Who will review my changes?

This repository is currently maintained by @openedx/wg-maintenance-edx-platform. Tag them in a comment and let them know that your changes are ready for review.

Where can I find more information?

If you'd like to get more details on all aspects of the review process for open source pull requests (OSPRs), check out the following resources:

When can I expect my changes to be merged?

Our goal is to get community contributions seen and reviewed as efficiently as possible.

However, the amount of time that it takes to review and merge a PR can vary significantly based on factors such as:

:bulb: As a result it may take up to several weeks or months to complete a review and merge your PR.

NiedielnitsevIvan commented 3 months ago

To fix pipelines, it is necessary to merge this PR in edx-ace and release a new version so that this PR can rely on it.

OmarIthawi commented 1 month ago

@NiedielnitsevIvan @GlugovGrGlib I see you're still actively pushing changes to this branch. Thank you for your efforts!

Is there anything I can help with to push this work forward? I know I've delayed other pull requests so I want to double check that I'm not blocking this one.

GlugovGrGlib commented 1 month ago

Hello @OmarIthawi with this latest commits we were able to address your comments from previous CR run and make all CI checks to pass. This PR is ready for the yet another review, hopefully final CR pass from your side.

OmarIthawi commented 3 weeks ago

@NiedielnitsevIvan thank you so much for going above and beyond. Please let me know when the tests are fixed so we can go ahead and merge it.

NiedielnitsevIvan commented 3 weeks ago

@NiedielnitsevIvan thank you so much for going above and beyond. Please let me know when the tests are fixed so we can go ahead and merge it.

@OmarIthawi Checks are fixed.

GlugovGrGlib commented 3 weeks ago

@bmtcril as this PR is finally approved, could you please check and merge this PR?

bmtcril commented 3 weeks ago

Thanks @GlugovGrGlib @NiedielnitsevIvan , I'm doing a last pass run through now. This seems to me to be a medium risk PR to land, so what I'd like to do is:

Thanks, this is looking great!

bmtcril commented 2 weeks ago

Merging on this is delayed while 2U works out issues on their side. Better to let the release queue clear than potentially get caught up in rollbacks and reverts while things get stable there. Discussion is happening in #cc-edx-platform if you'd like to follow along, but it seems likely it will take a day or two before we can merge. Sorry for the delay.

edx-pipeline-bot commented 2 weeks ago

2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production.

edx-pipeline-bot commented 2 weeks ago

2U Release Notice: This PR has been deployed to the edX production environment.

edx-pipeline-bot commented 2 weeks ago

2U Release Notice: This PR has been deployed to the edX production environment.