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

Feat: Implement whatsapp with vonage dart #246

Merged

Conversation

tejas-raskar
Copy link
Contributor

What does this PR do?

Implements whatsapp with vonage using dart

Test Plan

screen-20231022-003838.2.mp4

Related PRs and Issues

#200

Have you read the Contributing Guidelines on issues?

Yes

dart/whatsapp_with_vonage/lib/utils.dart Outdated Show resolved Hide resolved
dart/whatsapp_with_vonage/lib/main.dart Outdated Show resolved Hide resolved
dart/whatsapp_with_vonage/lib/main.dart Show resolved Hide resolved
dart/whatsapp_with_vonage/lib/main.dart Outdated Show resolved Hide resolved
dart/whatsapp_with_vonage/lib/utils.dart Outdated Show resolved Hide resolved
dart/whatsapp_with_vonage/lib/utils.dart Outdated Show resolved Hide resolved
@tejas-raskar
Copy link
Contributor Author

Hey @loks0n, I have made the necessary changes. Please review

dart/whatsapp_with_vonage/lib/main.dart Outdated Show resolved Hide resolved
dart/whatsapp_with_vonage/lib/main.dart Outdated Show resolved Hide resolved
dart/whatsapp_with_vonage/lib/main.dart Outdated Show resolved Hide resolved
@tejas-raskar
Copy link
Contributor Author

Hey @loks0n, changed the error handling to use JSON now.
Please review

Copy link
Member

@loks0n loks0n left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small final change before merge :)

Don't worry about the hacktoberfest deadline - we will add the hacktoberfest-accepted label before it arrives.

dart/whatsapp_with_vonage/lib/utils.dart Outdated Show resolved Hide resolved
dart/whatsapp_with_vonage/lib/utils.dart Outdated Show resolved Hide resolved
@tejas-raskar
Copy link
Contributor Author

Hey @loks0n,
Can we merge the PR before hacktoberfest so I can be eligible for the Appwrite swags?

@gewenyu99
Copy link

Hey there! There were a lot of big PRs during this Hacktoberfest, and we wanted to give everyone ample time to collaborate with our engineering team. If you were able to merge your PRs during October, amazing. If it’s still not merged, don’t worry about it either. Either way, we’ve got your Hacktoberfest swag minted and ready to ship.

Please comment with your Discord username here so we can contact you about your shipping information to deliver your Hacktoberfest swag.

@tejas-raskar
Copy link
Contributor Author

Hey there! There were a lot of big PRs during this Hacktoberfest, and we wanted to give everyone ample time to collaborate with our engineering team. If you were able to merge your PRs during October, amazing. If it’s still not merged, don’t worry about it either. Either way, we’ve got your Hacktoberfest swag minted and ready to ship.

Please comment with your Discord username here so we can contact you about your shipping information to deliver your Hacktoberfest swag.

Hey @gewenyu99, my discord username is tjsrskr

@gewenyu99
Copy link

Be in touch soon!

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

Successfully merging this pull request may close these issues.

3 participants