Skip to content
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

Fix for sending media #62

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Fix for sending media #62

wants to merge 3 commits into from

Conversation

sebastianarena
Copy link
Contributor

If not provided with the correct options in the parameters the methods
would crash trying to access the required properties.

I also upgraded the sendAudio method to be sendVoice since sendAudio is
for sending music.

If not provided with the correct options in the parameters the methods
would crash trying to access the required properties.

I also upgraded the sendAudio method to be sendVoice since sendAudio is
for sending music.
@shernshiou
Copy link
Contributor

I will need to test on this. BTW, I got your email, will reply when I have time.

@sebastianarena
Copy link
Contributor Author

Hi!
Sorry for not being more active here... the project that I run that uses the library has been in full development and I can't get my hands around what we talked about. Very sorry.

I did find another critical bug that I just fixed. Take a look.

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

Successfully merging this pull request may close these issues.

2 participants