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

Wrong 'ForwardCallIndicators' order in ISUP IAM description #174

Merged
merged 1 commit into from Nov 3, 2016

Conversation

Projects
None yet
3 participants
@agafox
Collaborator

agafox commented Nov 2, 2016

PR to fix tiny bug with wrong 'ForwardCallIndicators' order.

@deruelle deruelle added the Peer Review label Nov 2, 2016

@vetss vetss added the bug label Nov 2, 2016

@vetss vetss added this to the 8.0.0 milestone Nov 2, 2016

@agafox agafox closed this Nov 2, 2016

@deruelle deruelle removed the Peer Review label Nov 2, 2016

@agafox agafox reopened this Nov 2, 2016

@deruelle deruelle added the Peer Review label Nov 2, 2016

@agafox

This comment has been minimized.

Collaborator

agafox commented Nov 2, 2016

Hi, @vetss
You assigned this PR to me back. Should I do something with it?

@vetss vetss self-assigned this Nov 2, 2016

@vetss

This comment has been minimized.

Collaborator

vetss commented Nov 2, 2016

Hello @agafox

thanks for your update, I will check it ASAP

@vetss

This comment has been minimized.

Collaborator

vetss commented Nov 2, 2016

Hello @agafox

Please read THE TELESTAX OPEN SOURCE PLAYBOOK:
https://telestax.com/open-source/#Contribute
and sign Contributor License Agreement

We need you to sign a Contributor License Agreement for we can accept you contribution. Thank you.

@agafox

This comment has been minimized.

Collaborator

agafox commented Nov 2, 2016

@vetss
Done

@vetss vetss merged commit 3c4264b into RestComm:master Nov 3, 2016

@deruelle deruelle removed the Peer Review label Nov 3, 2016

@vetss

This comment has been minimized.

Collaborator

vetss commented Nov 3, 2016

Hello @agafox

I have added you commit into a master branch. Thanks for your work.

@agafox agafox deleted the agafox:isup_iam_bugfix branch Nov 3, 2016

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