openedx / openedx-app-android

The mobile app for Android for the Open EdX Platform.
Apache License 2.0
14 stars 19 forks source link

feat: [FC-0047] FCM #344

Closed volodymyr-chekyrta closed 2 weeks ago

volodymyr-chekyrta commented 3 weeks ago

This pull request introduces Firebase Cloud Messaging (FCM) to the project, enabling the functionality for receiving push notifications from Firebase.

Testing: • Verified push notifications are received when the app is in the foreground, background, and terminated states. • Confirmed the app correctly handles notification taps and displays relevant content. • Ensured token generation and handling work as expected, including handling token refreshes.

Please use the Firebase console for testing; here is a set of all the required fields for navigation:

Notifications:

  1. User has been enrolled notification_type: enroll course_id: Your course ID (example: course-v1:future+f1+2024)
  2. User has been unenrolled notification_type: unenroll
  3. User has been added as a beta tester notification_type: add_beta_tester course_id: Your course ID (example: course-v1:future+f1+2024)
  4. User has been removed from beta testers notification_type: remove_beta_tester course_id: Your course ID (example: course-v1:future+f1+2024)
  5. Someone left a response in the user discussion/question notification_type: forum_response topic_id: Your Topic ID (example: course) course_id: Your course ID (example: course-v1:future+f1+2024) thread_id: Your thread ID (example: 66584e96f2195c001ad219c7) comment_id: Your comment ID (example: 66585c9df2195c001ad219ce)
  6. Someone left a comment in the user discussion/question notification_type: forum_comment topic_id: Your Topic ID (example: course) course_id: Your course ID (example: course-v1:future+f1+2024) thread_id: Your thread ID (example: 66583d13f2195c001a56a182) parent_id: Your response ID (example: 66586579f2195c001ad219d8) comment_id: Your comment ID (example: 6659ac92f2195c001bf37f77)
openedx-webhooks commented 3 weeks ago

Thanks for the pull request, @volodymyr-chekyrta! Please note that it may take us up to several weeks or months to complete a review and merge your PR.

Feel free to add as much of the following information to the ticket as you can:

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.

Please let us know once your PR is ready for our review and all tests are green.

volodymyr-chekyrta commented 3 weeks ago

@dixidroid @omerhabib26, Kindly review the changes made in this pull request. Thank you 🙌

dixidroid commented 3 weeks ago

LGTM

HamzaIsrar12 commented 2 weeks ago

Reviewing ...

volodymyr-chekyrta commented 2 weeks ago

@HamzaIsrar12, thanks for your feedback, PR is ready for the next pass 🚀

openedx-webhooks commented 2 weeks ago

@volodymyr-chekyrta 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future.