Notifications active by default

Good idea. How shoul dthe interface for this look like? Would you only want to enable it for new users or also for existing ones?

An admin interface which would encompass my feature request http://drupal.org/node/15460 and this one is probably best implemented as an additional tab, probably labelled something like "users"

This would list each user and their settings for the notify module. It is probably easiest to use dropdown boxes if there are three settings and checkboxes for those where there are two settings (e.g. notification on/off)

It might also be a better idea to put this under admin/users, much like the profile.module interfaces with that module.

Any news on this update ?

Select * from users
See if user exists in notify list
If user does not exist, use one of the inserts below:

Title only
INSERT INTO `notify` (`uid`, `status`, `node`, `comment`, `attempts`, `teasers`) VALUES (1, 1, 1, 1, 0, 0);

Title + Teaser
INSERT INTO `notify` (`uid`, `status`, `node`, `comment`, `attempts`, `teasers`) VALUES (1, 1, 1, 1, 0, 1);

Title + Body
INSERT INTO `notify` (`uid`, `status`, `node`, `comment`, `attempts`, `teasers`) VALUES (1, 1, 1, 1, 0, 2);

http://01debug.com

I provided a patch for this issue here http://drupal.org/node/25215.

It does not provide an option to turn on notifications for all users. I can see how this might be helpful in some situations but I'm also very concerned about people using it on a live site where many users many have explicitly turned off notifications for their account.

If you could create a seperate patch and post here that would be great. How about adding a configuration option for the default new users. Perhaps create a new group in the settings which mirrors the user notify settings, which will then be applied to new users (if the settings is on). This would be instead of the default 1, 1, 1, 1 you have now. What do you think?

Here's a patch with the defaults hard-coded. If I get a chance over the next week or so I'll take a shot at adding the options on the settings page.

*bump*

That patch doesn't work on current versions of the module.

I am not a coder/patcher, but was able to get the defaults changed by doing this....

In notify.module, find
function notify_user

And, inside (on the next line) of this: switch ($type) {, insert this code:

case 'insert':
db_query('REPLACE {notify} (uid, status, node, teasers, comment) VALUES (%d, %d, %d, %d, %d)', $user->uid, 1, 1, 1, 1);
   break;

Need a re-roll against HEAD. We really just need a way to turn notify on for all users. I'm thinking for 4.7 we'll skip the mass enable for existing users as the operations hooks in D5 will make this easier. For 4.7 however, we should probably have a default on/off for new users in settings/notify or allow them to opt-in at registration http://drupal.org/node/39915.

I think that this functionality is covered by the patch here, and it will probably be easier to repatch that against HEAD. So I'm marking this issue a duplicate.