Update README.mdhttps://bellsystem35.com/%d8%aa%d8%b3%d9%84%d9%8a%d9%…#1
Update README.mdhttps://bellsystem35.com/%d8%aa%d8%b3%d9%84%d9%8a%d9%…#1bellsystem35 wants to merge 1 commit into
Conversation
…83-%d8%a7%d9%84%d9%85%d8%ac%d8%a7%d8%b1%d9%8a-%d8%a8%d8%a7%d9%84%d8%b6%d8%ba%d8%b7/
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 659c9f958a
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| <p style="margin: 0; color: #e2e8f0; font-weight: 800;">هذا النموذج يجهّز رسالة واتساب تحتوي على نوع المشكلة والمنطقة ومصدر الزيارة، مما يساعدك على معرفة العملاء القادمين من Google Ads أو البحث أو الزيارات المباشرة.</p> | ||
|
|
||
| </div> | ||
| <form style="background: #fff; border: 1px solid #dbeafe; border-radius: 34px; padding: 28px; box-shadow: 0 18px 48px rgba(15,23,42,.10);"> |
There was a problem hiding this comment.
When visitors use this new lead form, the submit button performs the browser's default form submission because the <form> has no onsubmit handler or action. The bsSubmitLeadForm(event) function above is therefore never called, so no WhatsApp message opens and no lead/conversion is logged from this form.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
Thanks for the review. You are correct: this form inside README.md does not have an "onsubmit" handler or an "action", so it would not submit correctly.
Since this code is in a GitHub README, JavaScript form handling is not reliable there. I will replace the form with direct contact links for phone, WhatsApp, and the official website, which is more suitable for GitHub README, SEO, and user experience.
I will update this section and then resolve the review comment.
| <a style="color: #fde68a; text-decoration: none; font-weight: 900;" href="tel:+971528913062">📞 052 891 3062</a> | ||
| | | ||
| <a style="color: #fde68a; text-decoration: none; font-weight: 900;" href="https://wa.me/971528913062?text=السلام%20عليكم،%20أريد%20خدمة%20تسليك%20مجاري%20في%20الشارقة" target="_blank" rel="noopener">💬 واتساب مباشر</a></div> |
There was a problem hiding this comment.
The new tracking helpers are not connected to the direct phone/WhatsApp CTAs, so clicks on these links navigate normally without calling bsTrackLeadClick, bsLogLead, or bsTrackGoogleAds. This means the advertised Google Ads conversion tracking and cooldown logic will be skipped for users who contact via the visible CTAs.
Useful? React with 👍 / 👎.
1 participant
…83-%d8%a7%d9%84%d9%85%d8%ac%d8%a7%d8%b1%d9%8a-%d8%a8%d8%a7%d9%84%d8%b6%d8%ba%d8%b7/