Skip to content
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

profile -> notifications for PM email #2038

Closed
illori opened this issue Jul 9, 2014 · 3 comments · Fixed by #2192
Closed

profile -> notifications for PM email #2038

illori opened this issue Jul 9, 2014 · 3 comments · Fixed by #2192
Labels
Milestone

Comments

@illori
Copy link
Contributor

illori commented Jul 9, 2014

there is a help popup stating that the settings for PM's is set via the my messages list. you can still set the checkbox for receive email for pm's here anyway.

if you go to your settings for pm's you can additionally set if you want to be emailed for pm's received. seems like only the option in the PM area controls when you get the email.

so the additional option for "When a personal message I sent gets replied to" does not seem to work.

@Oldiesmann
Copy link
Contributor

Looks like nobody remembered to code the appropriate background tasks for the options on the notifications page, though if we're going to put the options for board notifications there we should put the ones for PM notifications there as well.

@Oldiesmann Oldiesmann added the PM label Jul 11, 2014
@Oldiesmann Oldiesmann added this to the Beta 1 milestone Jul 11, 2014
@MissAllSunday
Copy link
Contributor

Do we really need a notification for an email? seems redundant to me, it would be enough to set a notification for a pm and when a pm is read.

@illori
Copy link
Contributor Author

illori commented Aug 12, 2014

the issue is the new notification page in the profile lists that you should get emails for pm's received, yet if you use those options you get no email and have to use the old setting in the pm area.

Dragooon added a commit to Dragooon/SMF2.1 that referenced this issue Aug 15, 2014
Not moving PMs to background task since it's
1) E-mail only
2) High priority enough to get handled instantly

Fixes SimpleMachines#2038

Signed-off-by: Shitiz Garg <mail@dragooon.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants