Implemented Discussions Feature

Aarush-Acharya commented 11 months ago

Description

Implemented Discussions Feature Allowing users to create discussions with act as scheduled rooms, also implementing notifications using FCM via firebase

Fixes #97

Type of change

[X] New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Tested it on Android (Currently push notifications only work for Android) Tested Discussion creation along with discussion subscriber management for the following action flows

[X] Create Discussions and experience discussion creator UI being enabled to delete discussion
[X] Subscribe and Unsubscribe to Discussions as non creator along with experience a different discussion UI as subscriber and non subscriber

https://github.com/AOSSIE-Org/Resonate/assets/92685647/57fe4ae8-f3e9-406a-b37c-f499b57d66ea

Testing Push notifications was done via creating a Discussion at a specific time and waiting in order to reach the time 5 minutes prior to scheduled time expecting to receive a push notification, testing was done with the following 3 action flows

[X] Sign In -> Subscribe to Discussion -> keep Signed In (Notification Received 👍🏻) worked as expected ✅
[X] Sign In -> Subscribe to Discussion -> Sign Out (Notification Not - Received 👎🏻) worked as expected ✅
[X] Sign In -> Subscribe to Discussion -> Sign Out -> Signs In Again (Notification Received 👍🏻 ) worked as expected ✅

The notifications have also been tested across the 3 states the App could be in

[X] Terminated works? ✅
[X] Background works? ✅
[X] Foreground works? ✅ (used flutter_local_notifications for implementing local notification after listening to notification sent via cron function using firebase_messaging)

Taps on notifications are also being handled ideally a tap on the notification should take you to the discussion screen with the discussion the notification mention being in focus, here is the status report for this feature

[X] Notification while in foreground ✅
[X] Notification while in background - to be done 🕐
[X] Notification while terminated - to be looked into not sure if possible 🤔

Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes
[X] Any dependent changes have been merged and published in downstream modules
[X] I have checked my code and corrected any misspellings

Maintainer Checklist

[ ] closes xxxx (Replace xxxx with the GitHub issue number)
[ ] Tag the PR with the appropriate labels

chandansgowda commented 10 months ago

Great job @Aarush-Acharya 🚀 This is indeed a great addition to the project. Thanks for your contribution.

Aarush-Acharya commented 10 months ago

Thanks @chandansgowda will contribute more and more

AOSSIE-Org / Resonate