We're currently using Flag to allow folks to add a session to their schedule. However, I've heard people ask how to implement features related to session signup such as:
- Email all presenters who are signed up for a specific session to let them know how to prepare
- Easy get a count of the number of people signed up for a session (and close signups) when a session reaches capacity - In fact it's the room that has a capacity, but sessions in a room could be made to inherit a similar capacity as a signup limit, if folks wanted to enforce this.
And I'm sure there are others.
I realize we could probably figure out a way to do the above in Flag, but it seems like Signup is really geared towards these features and others like them that folks are likely to want, and that it would be less work to instead work on #152659: Signup toggle link.
| Comment | File | Size | Author |
|---|---|---|---|
| #14 | 1115824-cod-signup-schedule-a.patch | 16.11 KB | ezra-g |
Comments
Comment #1
lisarex commentedI am +1 on this. Signup functionality seems much more useful.
Just need to bear in mind that sometimes people add two concurrent sessions to their schedule because they're undecided, so signup numbers won't be an accurate count *but* if those signed up are getting emails, then they might cancel their 2nd signup.
Comment #2
sketchysteven commented+1 from me... flags doesn't compile the list of attendees via views, even with relationships... I had to add the Flag Vista modules in order to compile a solid attendance list (of flagged users per session item). Works like a charm, but if signup will does this out of the box, let's use it.
Also, if signup will make each session selection/scheduling mutually exclusive, awesome! Hate to have folks register for multiples b/c they are undecided and "lock out" others b/c they create full sessions.
Comment #3
ezra-g commentedUpdating title to reflect intention here.
Comment #4
coltraneI'm not convinced this should replace the flag yet because of the complexities and assumptions Signup makes (such as sending a notification upon signup, and closing signups X hours before, etc). #152659: Signup toggle link can probably handle most of those, but moving to Signup here adds config management. I'd like to think on this more and finish the Signup link issue.
Comment #5
ezra-g commented> because of the complexities and assumptions Signup makes (such as sending a notification upon signup, and closing signups X hours before, etc
Signup has a global default that can be configured, and overridden on a per-node basis. It would be trivial to make the reminders default to off for session nodes.
It seems to me like a tradeoff between the complexity of signup module (which we're already using and provides the features folks have asked for) and adding views and potentially custom code to make Flag do everything that Signup already does.
Comment #6
coltraneThat is true for the email that is sent upon signup, but not for the reminder email. If the session node type does not have a date field associated for signup there should be no reminder though, correct?
Edit: I wouldn't define "trivial" to include code-level changes , which in this case there will at most have to be a form alter to keep the "Send confirmation" checkbox unchecked for all new session nodes, and at least a strongarmed variable. Or perhaps we leave it on?
Comment #7
ezra-g commentedThe signup reminder email is configurable on a per-node basis with the "Send reminder day(s) before this Event" option and the send_reminder column of the {signup} table.
Comment #8
coltraneOk, so the reminder email setting on the node only exists if the content type has been configured with a date to use. I think by default we should not use the time slot noderef (if we even could).
Comment #9
gregglesI see the concern about adding this to cod. We need to keep things super simple if we want them to be easy to use and signup is not historically all that simple in terms of the configuration screens.
That said, it seems like there's a lot of benefits to using signup and we can "just make signup simpler" like that link issue you mention which seems at least 90% ready to commit.
Our current alternative is "use flag and views and maybe some glue code" which doesn't feel all that much simpler.
So far the commenters on this issue are almost entirely in favor. I feel like we should move forward with it.
Comment #10
coltraneSignup writes the confirmation boolean to true in signup_save_node(), called from signup_nodeapi(). It looks like we can't hook in before it's written, the bool is taken from the default record in the signup table. Perhaps we could make the confirmation configurable per content type as part of Signup itself?
Comment #11
ezra-g commentedThat could be a good solution and a good extension to Signup. For cod, however, it would be super quick to set the confirmation mail setting in cod_session_nodeapi() case prepare, or using hook_form_alter(). Both seem like good solutions to me, though I slightly favor the latter.
Comment #12
coltraneEzra, please look at the code, I don't see how we can set a property of the node object that will override the send_confirmation bit. We'll have to do it afterwards or alter Signup. We also can't use hook_form_alter because the signup admin page would have to be saved, which is not in the std workflow of session submitters.
Comment #13
ezra-g commented@coltrane, I took a look at the code and saw what you're referring to, and filed a patch at #1182516: Signup overrides programmatically set signup values for new nodes. So, now we can add some code to disable signups on sessions that are not approved.
Comment #14
ezra-g commentedHere's a first pass at this.
Note that this patch to cod_support requires the the most recent patches to Signup from the following issues:
#1182516: Signup overrides programmatically set signup values for new nodes
#152659: Signup toggle link
Comment #15
ezra-g commentedAnother element of complexity this introduces is that the "cancel own signup" permission becomes necessary to edit ones own schedule, but this can also be used to cancel a signup on the main event.
Between this and the complexity added to the Signup module, I'm marking this as Postponed for Drupal 6 to be reconsidered for Drupal 7.
Comment #16
Bricks and Clicks Marketing@giftsketcher: Could you describe how you use Flag Vista to compile a list of users per session? Do you think it can be used to compile a list of attendees for a specific event too? Thanks.
Comment #17
mrconnerton commentedThe user stories for the request have been resolved in one form or another in the 7.x version.