DevoAbhi / WhatsSend-Frontend

Frontend of the WhatsSend App. It uses the api created in the WhatsSend-Backend repository.
https://whatssends.netlify.app/
1 stars 8 forks source link

Secure form Field #32

Closed Petsamuel closed 1 year ago

Petsamuel commented 2 years ago
netlify[bot] commented 2 years ago

Deploy Preview for whatssends ready!

Name Link
Latest commit c9fa62f2040eea12acd5c31185a7a271b332675f
Latest deploy log https://app.netlify.com/sites/whatssends/deploys/634bb1a442ab51000881bb09
Deploy Preview https://deploy-preview-32--whatssends.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

DevoAbhi commented 2 years ago

@Petsamuel Great work! I just have some points: 1) Messages are not being sent just after the user has scanned the qr. It will only be sent at the time mentioned by the user. Your success msg can misguide the user. 2)User can also enter long number for which user will get error msg "Invalid Receipient-Number/ Too Short". One thing you can do is to remove the Too Short part, Invalid Receipient-Number is enough.

Petsamuel commented 2 years ago

@Petsamuel Great work! I just have some points: 1) Messages are not being sent just after the user has scanned the qr. It will only be sent at the time mentioned by the user. Your success msg can misguide the user. 2)User can also enter long number for which user will get error msg "Invalid Receipient-Number/ Too Short". One thing you can do is to remove the Too Short part, Invalid Receipient-Number is enough.

Done.