New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

When verb is Dial, delay 200 OK to INVITE until outbound dial answers #1649

Closed
gvagenas opened this Issue Dec 14, 2016 · 9 comments

Comments

Projects
None yet
3 participants
@gvagenas
Collaborator

gvagenas commented Dec 14, 2016

No description provided.

@gvagenas gvagenas added this to the 8.1.0 milestone Dec 14, 2016

@gvagenas gvagenas self-assigned this Dec 14, 2016

@gvagenas gvagenas added the highlight label Jan 23, 2017

@gvagenas gvagenas closed this in 7975820 Jan 23, 2017

@gvagenas

This comment has been minimized.

Collaborator

gvagenas commented Jan 23, 2017

@kacmos please provide documentation for the feature

@gvagenas gvagenas reopened this Jan 23, 2017

@kacmos

This comment has been minimized.

Contributor

kacmos commented Jan 24, 2017

@gvagenas What kind of information do you need?
Sequence diagram with new call flows...?
Description of changes?

@gvagenas

This comment has been minimized.

Collaborator

gvagenas commented Jan 24, 2017

@kacmos if you can please create a document with the description of the feature, when to use, how to configure and 1-2 example call flows and important aspects of the feature, and then we can documentation writer to add it to the Restcomm docs.

@kacmos

This comment has been minimized.

Contributor

kacmos commented Feb 1, 2017

George,
I attach the document. Let me know if it's ok.
Issue1649 delay 200 ok to invite.docx

@gvagenas

This comment has been minimized.

Collaborator

gvagenas commented Feb 1, 2017

Thanks @kacmos , I will check document and let you know if anything needed.

@abdulazizali77

This comment has been minimized.

Contributor

abdulazizali77 commented Feb 2, 2017

@gvagenas Thanks, i will look into this

@abdulazizali77

This comment has been minimized.

Contributor

abdulazizali77 commented Feb 3, 2017

@gvagenas Apparently you had already merged the implementation for this enhancement by @kacmos on 16th and 19th Jan ? So im not sure why i was assigned this :D
I havent completely tested/verified the call flows yet, but what else is there to be done for this enhancement?

@gvagenas

This comment has been minimized.

Collaborator

gvagenas commented Feb 3, 2017

Sorry @abdulazizali77 , that was by mistake. Thanks

@abdulazizali77

This comment has been minimized.

Contributor

abdulazizali77 commented Feb 3, 2017

@gvagenas A good learning exercise nonetheless 👍

@gvagenas gvagenas closed this in 454d0b5 Feb 13, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment