search

PalisadoesFoundation / talawa-admin

Admin portal for the Talawa Mobile App. Click on the link below to see our documentation
https://docs.talawa.io/
GNU General Public License v3.0
130 stars 532 forks source link

MVP - Talawa Admin #1069

Closed palisadoes closed 10 months ago

palisadoes commented 10 months ago

We need to have an Talawa MVP by the end of the year. The MVP will be the master branch. To do this we will need to remove features that either don’t work or have insufficient functionality.

The features to remove include:

  1. Links to chat
  2. Notifications
  3. Dynamic linking

Other

  1. You need to work on removing these functionalities from this repo and work with the contributors below to make sure the correct functionality occurs in the other repos.
  2. Documentation related to configuring these features needs to be removed as they will be no longer valid.
  3. Test code coverage needs to be maintained. Coverage for the PR patches needs to be 100%

The plan is to:

Please work with @Ayush0Chaudhary (Talawa impacts), @EshaanAgg & @xoldyckk (Talawa-API impacts) @beingnoble03 (Talawa-Admin impacts). Ask to be assigned this task by them.

Issues related to this one include:

duplixx commented 10 months ago

Hey @Ayush0Chaudhary I can work with you in this milestone, Please let me know if you have any specific guidelines or instructions for this process.

palisadoes commented 10 months ago

@duplixx Please work with @beingnoble03 on this. He is very familiar with this repo.

duplixx commented 10 months ago

@palisadoes sure, hey @beingnoble03 please let me know if you have any specific instructions regarding the milestone. Thanks!!

duplixx commented 10 months ago

Hey @beingnoble03 , I've made progress on the removal of the dynamic linking to the chat section. I've commented out the relevant code and ensured that test coverage is maintained.

Regarding the notification functionality, could you please clarify what specific notifications are being referred to? Is it related to user notifications, system alerts, or something else?

noman2002 commented 10 months ago

@duplixx Comment out the notification which uses firebase i.e. user notification.

duplixx commented 10 months ago

@palisadoes @noman2002
All Talawa-Admin features are complete, except for user notifications. After an in-depth search, it seems this feature isn't talawa-admin repo.

Remove

image image image

palisadoes commented 10 months ago

@duplixx Thanks for that insight. Please proceed with the relevant PRs for this issue so that we can get the MVP live in the coming days.