Thanks for the issue, we will take it into consideration! Our team of engineers is busy working on many types of features, please give us time to get back to you.

To our amazing contributors: issues labeled bug are always up for grabs, but for feature requests, please wait until we add a ready for dev before starting to work on it.

If this is a feature request from an external contributor (not core team at Forem), please close the issue and re-post via GitHub Discussions.

To claim an issue to work on, please leave a comment. If you've claimed the issue and need help, please ping @forem-team. The OSS Community Manager or the engineers on OSS rotation will follow up.

For full info on how to contribute, please check out our contributors guide.

@PhilipHow if you are working on it, i'm happy to help. either dividing up the work, or helping with testing. I've been asking for this for a while.

@tonymet Are you free to take it on? If so, then I can assist. I won't be getting to this for a week or so.

@tonymet Are you free to take it on? If so, then I can assist. I won't be getting to this for a week or so.

I'll review the code this weekend and try to scope out the effort .

Here's my guess not knowing much about the code

1. UI & backend updates for settings
2. query mods (read-side) to filter the notifcations
3. possible write-side notification changes to reflect the updated settings.
4. updated tests
5. updated docs

Assuming dev env setup and the code points for the above are pretty clear, i'm happy to partner with you on it.

@tonymet Are you free to take it on? If so, then I can assist. I won't be getting to this for a week or so.

I started marking up the codepoints needing changes here : https://github.com/forem/forem/compare/main...tonymet:forem:new-post-notification-setting?expand=1

Take a look and let me know if you're up to dividing the work. I'll continue at a slow pace over the next week or so

@PhilipHow

I got a workign proof of concept on the branch above

You can test it out with

$ bin/rails db:migrate
$ bin/rails console
# set your user id
m= User.where(:id=>13).first
 m.notification_setting.new_post_notifications=false;
 m.notification_setting.save()

Then visit http://localhost:3000/notifications and notice the new-post notifs are gone.

What is Done

• new user setting notification_setting.new_post_notifications
• filter out new-post (article) notifications when new_post_notifications = false
• db migration to add column for above setting

What is next

• add UI crud for user setting
• test edge cases for filtering notification types (don't break @mentions)

@PhilipHow can you take a look at the PR and share your feedback? it's ~ 85% complete