[PR #532] [CLOSED] [FIX] Cypht: Check the phone number field to make sure it contains only numbers. If not, issue a warning to the user the user once. #879

Closed
opened 2026-02-25 21:36:30 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/cypht-org/cypht/pull/532
Author: @NasserNgandu
Created: 11/8/2021
Status: Closed

Base: masterHead: nasser_cypth


📝 Commits (2)

  • af2bac0 Check the telephone number field to ensure that it only contains numbers.
  • 7cf2a3f Update site.js

📊 Changes

2 files changed (+13 additions, -1 deletions)

View changed files

📝 modules/contacts/site.js (+12 -0)
📝 modules/local_contacts/modules.php (+1 -1)

📄 Description

…ers.

Pullrequest

Issues

  • None

Checklist

  • None

How2Test

  • None

Todo

  • None

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/cypht-org/cypht/pull/532 **Author:** [@NasserNgandu](https://github.com/NasserNgandu) **Created:** 11/8/2021 **Status:** ❌ Closed **Base:** `master` ← **Head:** `nasser_cypth` --- ### 📝 Commits (2) - [`af2bac0`](https://github.com/cypht-org/cypht/commit/af2bac0ab36228a8d690edc021424a53e350494e) Check the telephone number field to ensure that it only contains numbers. - [`7cf2a3f`](https://github.com/cypht-org/cypht/commit/7cf2a3fe1dd706beecccc2c2373f0342725dcf18) Update site.js ### 📊 Changes **2 files changed** (+13 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `modules/contacts/site.js` (+12 -0) 📝 `modules/local_contacts/modules.php` (+1 -1) </details> ### 📄 Description …ers. ## Pullrequest <!-- Describe the Pullrequest. --> ### Issues <!-- Which Issues does this fix, which are related? - fixes #XXX - relates #XXX --> - [X] None ### Checklist <!-- Anything important to be thought of when deploying? - [ ] Config Update - [ ] Breaking/critical change --> - [X] None ### How2Test <!-- Give a detailed description how to test your PR and confirm it is working as expected. --> <!-- Maintainers will check the Tests - [ ] Test1 - [ ] Test2 --> - [X] None ### Todo <!-- In case some parts are still missing, list them here. - [ ] Changelog --> - [X] None --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-25 21:36:30 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/cypht#879
No description provided.