@niteshsinha17 the changes that you have made in this PR were not expected.

Read this discussion to get a better idea of what is expected from the issue: #69 (review)

@niteshsinha17 the changes that you have made in this PR were not expected.

Read this discussion to get a better idea of what is expected from the issue: #69 (review)

Okay, I will check and will make changes according to it.

@pandafy from this #69 (review) I have concluded a few things that I wanted to share with you.

• I need to create that All setting which was removed in this PR but this time the need is to disable all notifications?
• If yes then we will again come into the confusing situation which was mentioned in the above PR.

if notifications are enabled in ALL but not enabled in the org, which setting overrides which?

If the above points are true then I think this could be a solution.

• That 'All' setting should not generate automatically. It will be created by the user by creating a new notification setting to disable all notifications.
• If a user disables notifications for all organizations for a particular type then we can check the record before sending a notification.
• If the user enables notifications both web and email for all organizations then we will delete that setting.

What do you think about it?

@nemesisdesign will be the right person to answer this query.

Closing as we're working on this in GSoC24. Thanks Nitesh!