Skip to content

Conversation

@alexanbj
Copy link
Contributor

@alexanbj alexanbj commented Jan 24, 2025

Denne PRen legger til første pakke for dette monorepoet, @obosbbl/format. Pakken skal etter hvert inneholde div formateringsmetoder for no og se.

Nå i første omgang inneholder pakken kun en metode, formatOrganiationNumber. Dette fordi den er veldig enkel. Det er litt fler regler for telefonnummer i feks Sverige.... Det skal gå fort å legge til fler her. Vil bare få ut en aller første versjon av pakken 😅

Pakken har to entrypoints, et for no og et for se. Tanken er at det skal være feature parity mellom de tilgjengelig formateringsmetodene til de forskjellige landene.

Jeg tenker at denne pakken ikke bør inneholde validering. Den vil bare forsøke å formatere inputen som best den kan. Validering kan vi løse i en egen pakke.

Bruk:

// 🇳🇴 example
import { formatOrganizationNumber } from '@obosbbl/format/no';

formatOrganizationNumber('000000000') // => '000 000 000'
formatOrganizationNumber('000-000-000') // => '000 000 000'
formatOrganizationNumber('   000 000 000') // => '000 000 000'

// 🇸🇪 example
import { formatOrganizationNumber } from '@obosbbl/format/se';

formatOrganizationNumber('0000000000') // => '000000-0000'

På sikt ser jeg også for meg et slags kombinert entrypoint, hvor du kan spesifisere locale som en del av metoden. Dette for å gjøre det enklere for de som har multitenant apper. Holder dette utenfor akkurat nå.

import { formatOrganizationNumber } from '@obosbbl/format';

formatOrganizationNumber('0000000000', { locale: 'se' }) // => '000000-0000'

Avklaring

Hvordan skal metodene navngis?

  • formatOrganizationNumber
  • formatPostalCode
  • formatPhoneNumber

Eller

  • organizationNumberFormat
  • postalCodeFormat
  • phoneNumberFormat

@alexanbj alexanbj requested review from a team, aulonm, kimknowit and theodorSchei January 24, 2025 13:23
Copy link
Contributor

@aulonm aulonm left a comment

Choose a reason for hiding this comment

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

nå er alle våre interne formatteringsfunskjoner i samme format som her, så for vår del funker det greit som det er nå.

@alexanbj alexanbj merged commit a4938b2 into main Jan 24, 2025
1 check passed
@alexanbj alexanbj deleted the format-package branch January 24, 2025 18:01
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