What did I do wrong?

I am forwarding calls to my cell when a caller presses 1 so I have it setup like this:

exten => 1,1,Background(pls-hold-while-try) ;if you press 1 you get my cell phone
exten => 1,2,Dial(SIP/voipdiscount/my cell number here)
exten => 1,3,Hangup

I tried to do this:

exten => 1,1,Background(pls-hold-while-try) ;if you press 1 you get my cell phone
exten => 1,2,MusicOnHold(default)
exten => 1,3,Dial(SIP/voipdiscount/my cell number here)
exten => 1,4,Hangup

When using the 2nd code snippet, it never rings my cell so I’m thinking there is a bit more I need to put in there. It just plays music forever. Can anyone help me out?

Thanks!

Instead of adding the MusicOnHold, why not just add music to the Dial? IE:
exten => 1,2,Dial(SIP/sip_provider/Cellnumber,m)
Hope this helps.

pressing ‘1’ will return the caller to extension 1, priority 1 in the current (or destination, if specified) context.

try this : have you incoming call start at the ‘s’ extension, and use

exten => s,1,Set(TIMEOUT(response)=7)
exten => s,2,Background(pls-hold-while-try) ;if you press 1 you get my cell phone
exten => t,1,Goto(s,1)
exten => 1,1,Dial(SIP/voipdiscount/my cell number here||m(default))
exten => 1,2,Hangup