Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Feature Request] Remove notification related implementation #2086

Closed
Ayush0Chaudhary opened this issue Nov 6, 2023 · 3 comments · Fixed by #2089
Closed

[Feature Request] Remove notification related implementation #2086

Ayush0Chaudhary opened this issue Nov 6, 2023 · 3 comments · Fixed by #2089
Assignees
@andoriyaprashant
Labels
feature request good first issue Good for newcomers

Comments

@Ayush0Chaudhary
Copy link

Is your feature request related to a problem? Please describe.
Currently it doesn't hinder the app but comment out all the notification related logic and add TODO:

Describe the solution you'd like
comment it out

Additional context
this is to make app compatible for MVP

Potential internship candidates
Please read this if you are planning to apply for a Palisadoes Foundation internship #359
@github-actions github-actions bot added the unapproved Unapproved, needs to be triaged label Nov 6, 2023
@Ayush0Chaudhary Ayush0Chaudhary removed the unapproved Unapproved, needs to be triaged label Nov 6, 2023
@andoriyaprashant
Copy link

Hello @Ayush0Chaudhary Please assign me this issue

@Ayush0Chaudhary
Copy link
Author

This is an urgent issue. Try to do it ASAP.

@andoriyaprashant andoriyaprashant mentioned this issue Nov 7, 2023
Merged
@andoriyaprashant
Copy link

Hello @Ayush0Chaudhary , as you mentioned, I have commented out the notification-related implementation part (#2089 ) . If you would like to make any changes, please let me know, and I'll try to make them as soon as possible.
Thank you

@palisadoes palisadoes mentioned this issue Nov 17, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@andoriyaprashant andoriyaprashant

Labels
feature request good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Notification Related Implementation Removed andoriyaprashant/talawa
2 participants
@Ayush0Chaudhary @andoriyaprashant