nxmndr I reviewed the PR you submitted on GitHub, and I don't see it as a suitable solution. You’re welcome to fork the project and use your version on your forum, but I believe the proposed change isn’t the right approach.
The design intentionally prevents members from disabling private notifications—a feature that’s particularly important for this extension. I don't think this fix effectively addresses the GDPR issue at hand.
Instead, my plan is to:
- Add an option in the notification types settings labeled “allow members to disable.”
- Create two separate notification blueprints to support this functionality.
The reasoning is that some forums use this extension for moderator alerts, while others rely on it to send welcome messages to new members. Naturally, we wouldn’t want users to be able to dismiss notifications like moderator alerts or important system messages.
Consider Instagram as an example: if your photo is removed due to a report, a notification must be sent, and users shouldn’t have the ability to disable such critical notifications.
For these reasons, I’m not going to accept this PR, and I recommend that you fork the extension if you wish to proceed with your changes.
