Hi there!
I´m recently trying up this module and I love it already! It´s just terrific the easy way that you can select your users to subscribe, according to your group.
I would like to know if can be done (or is it planned, or could be added...) some of these features:
1. The possibility for individual users to set inside their profiles if they want to be part of the "team" (even if they actually are members of the OG group).
2. The possibility to set all team member checkboxes, to "checked" by default.

Again, this module is fantastic. Thanks!!

Rosamunda

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Rosamunda’s picture

Meybe instead of "all checked by default" it could have a checkbox that allows to "check them all"?

cfennell’s picture

Status: Active » Patch (to be ported)
FileSize
1.86 KB

I needed a "checkall" option for a project that I'm working on. Here's an untested patch that does something similar; I'm sure the JS could be golfed a bit.

cfennell’s picture

Just occurred to me that I was focused only on node forms, so my selector was a bit too specific. Changed that...

cfennell’s picture

Ok, this is obviously a work in progress ;^), I don't update the hidden form value to reflect the checkall action in the above patches - I'll post something when I've done that.

cfennell’s picture

Ok, now the hidden "edit-notifications-team-selected" form element is updated when checkall is triggered. I also adopted some of the coding conventions of the module maintainer and was reminded of JQuery's "removeClass/addClass" functions as well, very handy. Anyway, it looks like this is going to work for my little project. Good luck on yours.

cfennell’s picture

Status: Patch (to be ported) » Needs review

Changing the status - not saying this should go in as-is, but some form of a "check all" feature would be great. Thanks.

David Goode’s picture

Hey all, I'll look at this shortly.

David

saveriuccio’s picture

@libsys,
I've just applied your patch and seems isn't working fine for me.. I'm really really interested on this feature.

Any suggestion?

David Goode’s picture

Status: Needs review » Needs work
FileSize
1.68 KB

Hey! I found a bit of a bug where it can add duplicate numbers/not remove items from the hidden form entry upon unchecking "check all." I fixed that by making the code simpler & just reusing the existing click() method. At this point I think the JS is fine, the only thing I'd like to see before committing this might be an option on the settings page for showing the check all button -- for now you could just make it a sitewide variable like the # of checkboxes setting. That settings page looked a little empty anyways :-). I'd disable it by default so people don't get any surprise functionality upon upgrading--otherwise known as bugs :-) Once that's done I'd be happy to commit, thanks for the work so far!

David

David Goode’s picture

Oh, also might be nice if you added a handler so that when everything was checked manually, check all would get checked automatically. That might be confusing though...either way, not required to be committed, but something to think about.

David