-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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.
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
🚀 Feature: Disable email notifications from settings #163
Comments
@nevo-david can you assign me this issue ? |
Hi @ishanarora04, can you let me know once you have successfully deployed your project on your computer? :) |
@nevo-david got it running on my local. |
Great, assigning this issue to you! |
@nevo-david also I saw your Upwork post for a job. Can you hire me ? |
Well if you manage to solve this issue, I don't see why not :) |
@nevo-david we are looking at Backend Changes only according to above ticket. right ? Also I am assuming we are just stopping the emails, rest all functionality like getting notifications in the notification box remains ? |
@nevo-david any update here ? |
Nope, it's both; why only the backend? @ishanarora04 |
can you assign this to me? @nevo-david |
It looks like @arafat4693 has done the work on this in #176 but the PR never got finished. It also looks like @ishanarora04 didn't have the time yet to submit changes, so unassigning from them. Let's wait a couple of weeks to see if @arafat4693 is willing to finish the PR. |
🔖 Feature description
Some users might want to opt out of getting emails, we need to allow them.
🎤 Why is this feature needed ?
Some people might not want to get transactional emails
✌️ How do you aim to achieve this?
NotificationService
we need to opt them out.settings.controller.ts
🔄️ Additional Information
Make sure it's in the User level and not in the Organization level.
👀 Have you spent some time to check if this feature request has been raised before?
Are you willing to submit PR?
None
The text was updated successfully, but these errors were encountered: