Skip to content

feat: #COCO-5031, mobile phone checks#937

Merged
pb-jo merged 2 commits intodevelop-b2schoolfrom
feat-COCO-5031-mobile-phone-checks
Dec 19, 2025
Merged

feat: #COCO-5031, mobile phone checks#937
pb-jo merged 2 commits intodevelop-b2schoolfrom
feat-COCO-5031-mobile-phone-checks

Conversation

@pb-jo
Copy link
Copy Markdown
Contributor

@pb-jo pb-jo commented Dec 12, 2025

Description

Utilisation de la lib libphonenumber de Google pour vérifier que le numéro de mobile saisi par l'utilisateur soit bien un numéro de mobile (certains pays ont besoin de FIXED_LINE_OR_MOBILE) et également que ce ne soit pas un numéro surtaxé.

Pas pu tester à 100% en local, car l'envoi de SMS n'est pas paramétrable (whitelist IP chez Sinch ?).

Fixes

#COCO-5031

Type of change

Please check options that are relevant.

  • Chore (PATCH)
  • Doc (PATCH)
  • Bug fix (PATCH)
  • New feature (MINOR)

Which packages changed?

Please check the name of the package you changed

  • admin
  • app-registry
  • archive
  • auth
  • cas
  • common
  • communication
  • conversation
  • directory
  • feeder
  • infra
  • portal
  • session
  • test
  • tests
  • timeline
  • workspace

Tests

  1. Describe here the tests you performed
  2. Step by step
  3. With expected results

Reminder

  • Security flaws

  • Performance impacts (think bulk !)

  • Unit tests were replayed

  • Unit tests were added and/or changed

  • I have updated the reminder for the version including my modifications

  • All done ! 😃

@pb-jo pb-jo requested review from jcbe-ode and vbillard91 December 12, 2025 11:56
@pb-jo pb-jo self-assigned this Dec 12, 2025
@pb-jo pb-jo force-pushed the feat-COCO-5031-mobile-phone-checks branch from 0828fde to 62ef364 Compare December 15, 2025 15:54
Copy link
Copy Markdown
Contributor

@jcbe-ode jcbe-ode left a comment

Choose a reason for hiding this comment

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

Pas de remarque particulière, ajouter des TUs me semble important aussi.

@sonarqubecloud
Copy link
Copy Markdown

@vbillard91 vbillard91 self-requested a review December 19, 2025 10:06
@pb-jo pb-jo merged commit 48c67a5 into develop-b2school Dec 19, 2025
1 check passed
@pb-jo pb-jo deleted the feat-COCO-5031-mobile-phone-checks branch December 19, 2025 14:07
jenkinsEdificePublic pushed a commit that referenced this pull request Dec 22, 2025
* feat: #COCO-5031, mobile phone checks

* Feedbacks
jenkinsEdificePublic pushed a commit that referenced this pull request Jan 6, 2026
* feat: #COCO-5031, mobile phone checks

* Feedbacks
pb-jo added a commit that referenced this pull request Jan 8, 2026
* feat: #COCO-5031, mobile phone checks

* Feedbacks
jenkinsEdificePublic pushed a commit that referenced this pull request Jan 12, 2026
* feat: #COCO-5031, mobile phone checks

* Feedbacks
jenkinsEdificePublic pushed a commit that referenced this pull request Feb 3, 2026
* feat: #COCO-5031, mobile phone checks

* Feedbacks
vbillard91 pushed a commit that referenced this pull request Feb 6, 2026
* feat: #COCO-5031, mobile phone checks

* Feedbacks
jenkinsEdificePublic pushed a commit that referenced this pull request Feb 6, 2026
* feat: #COCO-5031, mobile phone checks

* Feedbacks
jcbe-ode pushed a commit that referenced this pull request Feb 10, 2026
* feat: #COCO-5031, mobile phone checks

* Feedbacks
damienromito pushed a commit that referenced this pull request Feb 16, 2026
* feat: #COCO-5031, mobile phone checks

* Feedbacks
jenkinsEdificePublic pushed a commit that referenced this pull request Feb 16, 2026
* feat: #COCO-5031, mobile phone checks

* Feedbacks
jenkinsEdificePublic pushed a commit that referenced this pull request Feb 17, 2026
* feat: #COCO-5031, mobile phone checks

* Feedbacks
benjaminperez pushed a commit that referenced this pull request Feb 19, 2026
* feat: #COCO-5031, mobile phone checks

* Feedbacks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants