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

remove pom.xml~, update guide1 pom.xml to align with the trunk, avp.getOctetString() #42

Merged
merged 1 commit into from May 29, 2016

Conversation

Projects
None yet
3 participants
@phillipcheng
Contributor

phillipcheng commented May 27, 2016

update guide1 pom.xml to align with the trunk

value = avp.getOctetString(); to value = new String(avp.getOctetString(), StandardCharsets.UTF_8);

remove pom.xml~
update guide1 pom.xml align with the trunk
convert byte[] to String updated
@deruelle

This comment has been minimized.

Member

deruelle commented May 27, 2016

Thanks for the pull request @phillipcheng.

Can you please sign the CLA at https://telestax.com/open-source/#Contribute so we can accept your contribution ?

@deruelle deruelle added this to the 1.7.0.FINAL milestone May 27, 2016

@phillipcheng

This comment has been minimized.

Contributor

phillipcheng commented May 27, 2016

I signed.

2016-05-27 11:46 GMT-07:00 Jean Deruelle notifications@github.com:

Thanks for the pull request @phillipcheng
https://github.com/phillipcheng.

Can you please sign the CLA at
https://telestax.com/open-source/#Contribute so we can accept your
contribution ?


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#42 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AHhpfMuCIFxBdLwsB8sn0QXWSlZlcFVhks5qFzwcgaJpZM4Iow1g
.

@deruelle deruelle merged commit dfde497 into RestComm:master May 29, 2016

@deruelle

This comment has been minimized.

Member

deruelle commented May 29, 2016

Thanks @phillipcheng. contributions acknowleged at https://telestax.com/acknowledgements/

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