8000 V2 `whatsapp` by michaelgatuma · Pull Request #437 · finsweet/attributes · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

V2 whatsapp #437

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

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

V2 whatsapp #437

wants to merge 14 commits into from

Conversation

michaelgatuma
Copy link
Contributor

No description provided.

@vercel
Copy link
vercel bot commented Aug 22, 2023

@michaelgatuma is attempting to deploy a commit to the Finsweet Team on Vercel.

To accomplish this, @michaelgatuma needs to request access to the Team.

Afterwards, an owner of the Team is required to accept their membership request.

If you're already a member of the respective Vercel Team, make sure that your Personal Vercel Account is connected to your GitHub account.

Copy link
Contributor
@robertkibet robertkibet left a comment

Choose a reason for hiding this comment

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

Great stuff 🚀

Just some questions and a possible code improvement

name: 'Button',
description: 'A button to open WhatsApp',
allowedTypes: ['Link'],
settings: [SCHEMA_SETTINGS.phone, SCHEMA_SETTINGS.message],
Copy link
Contributor

Choose a reason for hiding this comment

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

should we have another element without the settings? for dynamic elements this button is just a wrapper and has no fs-whatsapp-phone or fs-whatsapp-message
🤔

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Maybe @alexiglesias93 can advise on this.

If user accidentally adds a number like ++1233434, this regex will not remove the extra plus. Therefore, remove everything that is not a number from the phone number, then append a '+' sign
robertkibet
8000 robertkibet previously approved these changes Aug 30, 2023
Copy link
Contributor
@robertkibet robertkibet left a comment

Choose a reason for hiding this comment

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

looks good

@robertkibet
Copy link
Contributor

@michaelgatuma just one minor thing I noticed:

image

When testing this locally, the resultant whatsapp message seems to add some white spaces, as shown by the multiple ++++++++++++++++++++++++ appended to the href, Whatsapp by default tries to trims this automatically, but its not super successful in the end.

image
<a
  fs-whatsapp-element="button"
  href="https://wa.me/+254725353920?text=%0A++++++++++++++++++++++++Wassup%2C+I%27m+interested+in+a+quote%21%0A++++++++++++++++++++++++%0A++++++++++++++++++++++++Are+you+interested%3F+visit+mywebsite.com%0A++++++++++++++++++++++"
  class="button w-inline-block"
  target="_blank"
  ><img
    src="https://assets.website-files.com/64af195b0306c12d295ea1dc/64af1bbcf71d8da4ffa20b98_whatsapp-color-svgrepo-com.svg"
    loading="lazy"
    alt=""
    class="icon-1x1-medium"
  />
  <div>Total Body Car Repair</div>
  <div>Contact us on WhatApp</div>
  <div fs-whatsapp-element="message" class="hide">
    Wassup, I'm interested in a quote!
    <br />
    Are you interested? visit mywebsite.com
  </div>
  <div fs-whatsapp-element="phone" class="hide">+254725353920</div>
</a>

@alexiglesias93 alexiglesias93 added this to the 2024 Q3 milestone Jan 16, 2024
Base automatically changed from v2 to master March 21, 2024 18:41
@alexiglesias93 alexiglesias93 dismissed robertkibet’s stale review March 21, 2024 18:41

The base branch was changed.

Copy link
vercel bot commented Aug 15, 2024

An owner of the Finsweet Team on Vercel declined @michaelgatuma's request to join.

In order for their commit to be deployed, @michaelgatuma must push and request access again.

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.

4 participants
0