DTMF support, native Go error wrapping, code cleanup, fixed example

[florentchauveau]

No description provided.

[florentchauveau]

@Ulexus let me know if you want any change in the PR 🙏

[Ulexus]

Thanks! Lots of good stuff in here.

[Ulexus]

If you want to update the README, I think that's my only nit-ish objection. Thanks

[florentchauveau]

If you want to update the README, I think that's my only nit-ish objection. Thanks

Done! I have removed the diff lines and left the note about DTMF.

I will try to get the Asterisk patch merged now 😃

[florentchauveau]

Hold on before merging please, I am working on the Asterisk PR, so the link to the patch might change.

[Ulexus]

@florentchauveau are we still waiting for your Asterisk PR update, or is this ready to merge?

[florentchauveau]

@florentchauveau are we still waiting for your Asterisk PR update, or is this ready to merge?

The PR is getting discussed, I have fixed all of the reviewer comments. It took a bit of time to get it reviewed by someone.

If it's not merged by the end of next week, I'll just provide a link to the branch on my repo, and we'll update the README again once the PR is merged? what do you think?

[Ulexus]

Either way is fine; I'm not in a rush. I just saw that you had updated this PR since the last message, so I wanted to make sure you weren't waiting on me.

[florentchauveau]

Either way is fine; I'm not in a rush. I just saw that you had updated this PR since the last message, so I wanted to make sure you weren't waiting on me.

The PR has been merged into master 🎉

I've updated the README as well. Thank you!

[florentchauveau]

Also, I think this library is stable enough for a 1.0.0 release tag. But I'll let you decide on this!