Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: se soluciona problemas con sendContacts para el provider meta #909

Merged
merged 4 commits into from
Nov 10, 2023

Conversation

jorgechavarriaga
Copy link
Collaborator

se corrige el nombre phone a phones (plural) y se pasa por el map para recibir los phones. se adicionan todos los campos soportados por meta (addresses, emails, urls, org, etc)

Que tipo de Pull Request es?

  • [ X] Mejoras
  • [ X] Bug
  • Docs / tests

Descripción

Al utilizar sendContacts con el provider META no funcionaba.

        phone: [
            {
                phone: contact.phone,
                wa_id: contact.phone,
                type: 'MOBILE',
            },
        ],

De acuerdo a la documentacion de META, es phones (en prlural) y acepta multiples numeros de telefonos, por lo que debe de quedar:

        phones: contact.phones.map((phone) => ({
            phone: phone.phone,
            wa_id: phone.wa_id,
            type: phone.type,
        })),

Igualmente, de acuerdo a la documentacion de META, se adiciona soporte para enviar en el contacto la informacion de
addresses, birthday, emails, org, urls.

Ahora se pueden enviar correctamente contactos con el proveedor META y se visualiza asi:

Screenshot 2023-11-09 at 5 07 17 PM

Forma parte de este proyecto.

leifermendez and others added 4 commits November 1, 2023 10:15
next-version-0.1.34
release-0.1.34
se corrige el nombre phone a phones (plural) y se pasa por el map para recibir los phones.
se adicionan todos los campos soportados por meta (addresses, emails, urls, org, etc)
@leifermendez leifermendez changed the base branch from main to dev November 10, 2023 09:04
@leifermendez leifermendez merged commit b148060 into codigoencasa:dev Nov 10, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants