I cant see any remind me button to send a reminder to people who havent joined..

pls help...do i have to activate it somewhere?

thanks

CommentFileSizeAuthor
#11 invite_reminder_4.patch15.84 KBsmk-ka
#6 invite_reminder_2.patch16.13 KBsmk-ka
#5 invite_reminder_1.patch15.74 KBsmk-ka
#3 invite_reminder_0.patch15.2 KBsmk-ka
#1 invite_reminder.patch27.78 KBsmk-ka

Comments

smk-ka’s picture

smk-ka commented
Title: No Remind Me button » Add Reminder button
Version: 5.x-1.9 » 5.x-1.x-dev
Component: User interface » Code
Assigned: Unassigned » smk-ka
Category: support » feature
Priority: Critical » Normal
Status: Active » Needs review
StatusFileSize
new invite_reminder.patch27.78 KB

It simply wasn't implemented yet. Luckily, I had some some time to add the reminder button, but it turned out to require quite some changes to the existing code.

This is why I need your help to test the patch thoroughly. Play around with the new 'Editable subject' setting, send invites from the invite page and the block, and test different user roles. Then edit the database records and remove the first '1' from the expiry column. Now you should be able to send reminders.
--
Stefan Kudwien
unleashed mind

gurukripa’s picture

gurukripa commented

my site is currently live..and unfortunately there are some problems in getting the test site up on my local machine..some issues..it needs an upgrade i think...

will try this when it does get alright..

am not too techie..so afraid to play with database..since i cant troubleshoot :)

pls let me know if this feature is working fine.

thanks

smk-ka’s picture

smk-ka commented
StatusFileSize
new invite_reminder_0.patch15.2 KB

Rerolled the patch against HEAD. Still needs in-depth testing, so still not production-ready, sorry.

smk-ka’s picture

smk-ka commented
Status: Needs review » Needs work

Found an issue: the invite code changes upon sending a reminder email, which invalidates the original code. Ideally it doesn't, so the invitee could also join using the link in the original invitation.

smk-ka’s picture

smk-ka commented
Status: Needs work » Needs review
StatusFileSize
new invite_reminder_1.patch15.74 KB

New patch fixing above issue.

smk-ka’s picture

smk-ka commented
StatusFileSize
new invite_reminder_2.patch16.13 KB

Another fix: hook_invite($op = 'invite') shouldn't be triggered when sending a reminder, only on the initial invitation.

gurukripa’s picture

gurukripa commented

Hi.
Is it possible that you could incorporate the patch for remind in the Dev version..and then we cld try it out.
Thanks
:)

smk-ka’s picture

smk-ka commented

Rerolled patch against latest DRUPAL-5 branch.

@gurukripa
Sorry, but this is not the only patch to be reviewed. If I would apply a patch as large as this one, it would almost certainly break other patches in the queue. Besides, applying patching isn't difficult at all.

gurukripa’s picture

gurukripa commented

I have a suggestion.
It would be nice to have Remind all who have not Joined...also as an option.
This wld make things easier than having to click a few hundred times to invite each person again.

Also a Bulk delete invitations option cld also help.

Thanks for this nice module.

Anonymous’s picture

Anonymous (not verified) commented

I'm very confused - I'm trying to get this working but the patch fails on the latest stable version, the latest dev and head. I've even gone through and added the patch manually but the remind tab doesn't show up, and when it does, it just shows the invite page.

Is there any way I can get this working? If so, can u tell me which version of invite I need to use and how I can apply the patch. Thanks

Also, some people on my site have thousands of invites they want to remind people of so I second the comment about remind all function.

smk-ka’s picture

smk-ka commented
StatusFileSize
new invite_reminder_4.patch15.84 KB

New patch against current DRUPAL-5 branch.

Yet, I'm still unsure how to satisfy everyone. Your point about a bulk reminder is valid (vs. a resend option that allows further customization, which is what this patch implements). Couldn't the bulk reminder also sit in a separate module? It seems to me you don't need the invitation tracker at all, since you don't want to selectively resend invites, but just want to send a reminder to anyone whose invitation is more than X days old (possibly even by cron, so except a settings page there would be no UI required at all). What should the reminder look like? Just resend the original invitation? Or another e-mail template saying something like 'Hey, this is just a reminder...'?

smk-ka’s picture

smk-ka commented
Version: 5.x-1.x-dev » 5.x-2.x-dev
Status: Needs review » Fixed
Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.