[Home]

Summary:ASTERISK-05961: [patch] Temporary Greeting reminder at login
Reporter:Kyle Haefner (khaefner)Labels:
Date Opened:2006-01-03 10:54:43.000-0600Date Closed:2006-05-05 16:23:47
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Applications/app_voicemail
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) temp_greeting_reminder_app_voicemail.c.diff
( 1) temperary_greeting_warn_app_voicemail.c.diff
( 2) temp-greeting-reminder_app_voicemail.c.diff
( 3) vm-tempgreetactive.gsm
( 4) voicemail.conf.diff
( 5) voicemail.conf.sample.diff
Description:Remind the user at login that their temperary greeting is set.
Give the option to press 1 to continue (leave temperary greeting set) or 2 to delete the greeting.  

Requires additional prompt
vm-tempset:
"Your temperary greeting is currently set, press 1 to continue, 2 to remove"

I can provide the promt for testing if needed.

This modifies the vm_intro_en section of app_voicemail.c
Comments:By: Jason Parker (jparker) 2006-01-22 00:39:33.000-0600

This should be a setting in voicemail.conf.  I don't think we want to remind everybody every time.
By: Kyle Haefner (khaefner) 2006-01-23 08:37:16.000-0600

North you are right, I'll change the code and re-upload.
By: Kyle Haefner (khaefner) 2006-01-23 16:14:10.000-0600

Uploaded new patch for app_voicemail.c
This adds checking voicemail.conf file for a line
tempgreetset=yes/no
if the line is not there it defaults to no.
Also attatched is a diff for the voicemail sample that shows the
config line.
If anyone has suggestions for a variable other than tempgreetset I'm open
to suggestions.
By: Kevin P. Fleming (kpfleming) 2006-02-14 16:52:36.000-0600

I would call the option 'tempgreetwarn', since that is what it does. I would also _not_ add the ability to change it at that point, just go ahead and let them into their mailbox after playing this message. It's easy enough to delete the greeting the normal way.

Will you get the required prompt for this patch and resubmit? Thanks.
By: Clod Patry (junky) 2006-04-11 22:56:51

Do we have the prompts so we can modify vm_intro_x, where x is any other language.
By: Serge Vecher (serge-v) 2006-05-03 10:18:11

Updating title since khaefner has posted the prompts.
By: Russell Bryant (russell) 2006-05-05 16:23:46

added to the trunk in revision 25057, thanks!