🚀 Feature: Disable email notifications from settings

[nevo-david]

trafficstars

🔖 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?

• Create a new field in the table User called emailNotifications with a default of true!
• If it's false, in every function in NotificationService we need to opt them out.
• Add their feature inside of Settings in the system
• Add the crud for it inside of 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?

• [X] I checked and didn't find similar issue

Are you willing to submit PR?

None

[ishanarora04]

@nevo-david can you assign me this issue ?

[nevo-david]

Hi @ishanarora04, can you let me know once you have successfully deployed your project on your computer? :)

[ishanarora04]

@nevo-david got it running on my local.

[nevo-david]

Great, assigning this issue to you!

[ishanarora04]

@nevo-david also I saw your Upwork post for a job. Can you hire me ?

[nevo-david]

Well if you manage to solve this issue, I don't see why not :)

[ishanarora04]

@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 ?

[ishanarora04]

@nevo-david any update here ?

[nevo-david]

Nope, it's both; why only the backend? @ishanarora04

[nikhilrajbhar17]

can you assign this to me? @nevo-david

[jamesread]

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.

[bansal-harsh-2504]

@jamesread Can I attempt?

[jamesread]

We're still waiting for @arafat4693

[hackit-coder]

Can I work on this?

[arafat4693]

I fixed the bug long time ago, however, no one merged it

[jamesread]

@arafat4693 it's not finished, the code isn't acceptable at the moment - see Nevo's comments on the PR.