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(createbot.ts, createbot.types.ts, messages.types.ts): support fo… #5

Merged
merged 2 commits into from
Oct 14, 2024

Conversation

RyzorBent
Copy link
Contributor

…r product and catalog messages

Add support for product and catalog messages

  • Implement new functions: sendProduct, sendProductList, and sendCatalog
  • Enable sending interactive product and catalog messages via WhatsApp Cloud API
  • Update MessageSender interface to include new functions
  • Add corresponding types for product and catalog messages
  • Update README with new features and API usage documentation
  • Enhance library capabilities for e-commerce use cases

Description of change

Pull-Request Checklist

  • Code is up-to-date with the main branch
  • npm run lint passes with this change
  • npm run test passes with this change
  • This pull request links relevant issues as Fixes #0000
  • There are new or updated unit tests validating the change
  • Documentation has been updated to reflect this change
  • The new commits follow conventions outlined in the conventional commit spec

…r product and catalog messages

Add support for product and catalog messages- Implement new functions: sendProduct, sendProductList,
and sendCatalog- Enable sending interactive product and catalog messages via WhatsApp Cloud API-
Update MessageSender interface to include new functions- Add corresponding types for product and
catalog messages- Update README with new features and API usage documentation- Enhance library
capabilities for e-commerce use cases
@RyzorBent
Copy link
Contributor Author

Realised that i missed an entry in src/expressRouter.ts --> case 'order':. Pushing a fix

@RyzorBent
Copy link
Contributor Author

Hi @j05u3, Any feedback on this pull req, is there perhaps something i'm missing that i need to correct?

@j05u3
Copy link
Owner

j05u3 commented Oct 2, 2024

Sorry, I missed your comment @RyzorBent. I just took a look at your code and it looks good 👍. If you tested it locally or in some way maybe you can send some screenshots of it working 🙏, otherwise if you tell me that you are certain that it works then we can merge it I'd say 👍

@RyzorBent
Copy link
Contributor Author

RyzorBent commented Oct 2, 2024

Morning @j05u3, ... here is a screen cast of the bot working with new code, everything should work fine :)
https://drive.google.com/file/d/1v9rH9yjqPVJWP5KYbugAgdHAujXwIpwC/view?usp=drive_link

@j05u3
Copy link
Owner

j05u3 commented Oct 3, 2024

Hi @RyzorBent, I just requested asked 🙏

@j05u3
Copy link
Owner

j05u3 commented Oct 3, 2024

Hi @RyzorBent , I just saw the video, it looks good. Let's merge then 🙌🎉

@RyzorBent
Copy link
Contributor Author

Hi @j05u3,I think you can merge on your side. as i don't have powers to merge. 🤗

@j05u3
Copy link
Owner

j05u3 commented Oct 14, 2024

Oh, I'm sorry, will do now then 🙏🙌

@j05u3 j05u3 merged commit 5e6cd12 into j05u3:main Oct 14, 2024
2 checks passed
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