Skip to content
This repository has been archived by the owner on Mar 22, 2023. It is now read-only.

Improve ivr routing for phone dialins. #137

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

liske
Copy link
Contributor

@liske liske commented May 2, 2020

I had some problems with the default dialplan extension for phone callins:

  • if the caller enters a 5 digit pin but no conference was found by the pin he gets dropped
  • if the caller fails to enter a valid pin (read: 5 digits) three times in a row he got dropped, too
  • if the caller enters the correct pin he is directly placed into the conference without any further indication which might feel irritating

The patch changes the ivr sequence this way:

  • BEGIN: play the conf-pin.wav only once
  • query the pin, on syntax errors use conf-bad-pin.wav to ask the caller to try it again
  • after three failures transfer the call to BEGIN again
  • if a pin was entered and a conference has been found:
    • playback conf-welcome.wav and join the caller into the conference
  • if a pin was entered and no conference has been found:
    • playback conf-bad-pin.wav
    • transfer to BEGIN again

IMHO this improves the usability for dialins since the caller is no more silently dropped or transfered.

@Sea444
Copy link

Sea444 commented May 8, 2020

I like you're PR, but isn't it wrong at this place? This is for documentation.
I think your PR belongs to https://github.com/bigbluebutton/bigbluebutton/pulls. Could you correct this?

And maybe you could have a look at this Freeswitch-dialplan issue: bigbluebutton/bigbluebutton#9161?

@basisbit
Copy link
Contributor

basisbit commented May 8, 2020

this PR edits the old BBB 1.x install documentation and thus should not be merged. cc @liske

@liske
Copy link
Contributor Author

liske commented May 8, 2020

this PR edits the old BBB 1.x install documentation and thus should not be merged. cc @liske

So where do I find the 2.2 documentation sources? The _pages/install.md is just a symlink to the the modified file.

@ffdixon
Copy link
Member

ffdixon commented May 8, 2020

Check out the file 2019-02-15-install.md -- I'll update the symbolic links so they are pointing at the right items.

@liske
Copy link
Contributor Author

liske commented May 8, 2020

The correct file is 2019-02-14-customize.md, isn't it? I've updated and rebased the PR.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants