Skip to content

Conversation

@alexanbj
Copy link
Contributor

@alexanbj alexanbj commented Jan 25, 2025

Denne PRen legger til støtte for å formatere telefonnummer.

import { formatPhoneNumber } from '@obosbbl/format/no';

 formatPhoneNumber('00000000') // => '00 00 00 00'
 formatPhoneNumber('80000000') // => '800 00 000'

import { formatPhoneNumber } from '@obosbbl/format/se';

 formatPhoneNumber('07012345678') // => '070-123 45 678'
 formatPhoneNumber('0812345') // => '08-123 45'
 formatPhoneNumber('0311234567') // => '031-123 45 67'
 formatPhoneNumber('0303123456') // => '0303-12 34 56'

For Norge er regelen veldig enkel. Det er bare to typer formatering:

  • 00 00 00 00 for vanlige nummere
  • 800 00 000 for såkalte 800-numre.

Sverige har et ganske mye mer avansert system, med en area code som kan være 2, 3 eller 4 sifre 😓

https://en.wikipedia.org/wiki/National_conventions_for_writing_telephone_numbers

Screenshot 2025-01-26 at 21 58 48

Det er dog mulig å lage regler for det, da det bare finnes en area kode på 2 siffer, og 20 stk som er 3 siffer, mens resten er 4. Se https://en.wikipedia.org/wiki/Telephone_numbers_in_Sweden

Dersom denne godkjennes lager vi den første releasen av denne pakken. Trenger disse metodene på min side 😅

@alexanbj alexanbj requested review from a team and removed request for gabrielwallin January 26, 2025 21:07
@alexanbj alexanbj marked this pull request as ready for review January 26, 2025 21:07
@aulonm
Copy link
Contributor

aulonm commented Jan 26, 2025

skal se på denne mer imorgen, men bør vi ha formattering med og uten +46/+47? Fordi i sverige så forosvinner 0-tallet hvis det er skrevet med +46. Vi har det i Norge hvor vi viser +47 og skulle ønske vi hadde det med sverige også men ikke orka styre med prefix håndteringen 😅

@alexanbj
Copy link
Contributor Author

skal se på denne mer imorgen, men bør vi ha formattering med og uten +46/+47? Fordi i sverige så forosvinner 0-tallet hvis det er skrevet med +46. Vi har det i Norge hvor vi viser +47 og skulle ønske vi hadde det med sverige også men ikke orka styre med prefix håndteringen 😅

Ja. Det er nok sikkert greit å få inn noe landekode-greier på sikt. Men da tenker du på for user submitted input? For ut fra tjenestene våre kommer vel landekode som en egen property?

@aulonm
Copy link
Contributor

aulonm commented Jan 27, 2025

skal se på denne mer imorgen, men bør vi ha formattering med og uten +46/+47? Fordi i sverige så forosvinner 0-tallet hvis det er skrevet med +46. Vi har det i Norge hvor vi viser +47 og skulle ønske vi hadde det med sverige også men ikke orka styre med prefix håndteringen 😅

Ja. Det er nok sikkert greit å få inn noe landekode-greier på sikt. Men da tenker du på for user submitted input? For ut fra tjenestene våre kommer vel landekode som en egen property?

Litt av begge deler men mest bare visning og ikke ved input. Det kommer som egen property ja men vi liker å vise hele nummeret med +47-prefix og da blir det feil å vise svensk nummer slik dtp har det lagra (de har 0-prefix)

Copy link

@gabrielwallin gabrielwallin left a comment

Choose a reason for hiding this comment

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

feature requests: ikke la 08-nummer få tillgång 😤

@alexanbj alexanbj merged commit 0f78312 into main Jan 27, 2025
1 check passed
@alexanbj alexanbj deleted the formatPhoneNumber branch January 27, 2025 10:41
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.

4 participants