Skip to content

Conversation

@aulonm
Copy link
Contributor

@aulonm aulonm commented Apr 9, 2025

implementerer bare for norge da sverige har 100 forskjellige måter å validere kontonummer

sykt enkelt når det først var satt opp mod11 funksjon :D

usikker på navngiving:

  • bankAccountNumber
  • accountNumber (lener meg mot denne egentlig)

Vil vi har formatering? tillate feks 12345 67 8903 eller 12345.67.8903

@aulonm aulonm requested a review from alexanbj April 9, 2025 19:15
Copy link
Contributor

@alexanbj alexanbj left a comment

Choose a reason for hiding this comment

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

Ser veldig greit ut! Bare noen små nits her og der

@alexanbj
Copy link
Contributor

Vil vi har formatering? tillate feks 12345 67 8903 eller 12345.67.8903

Hvis du ser på de andre metodene har det optiona options parameter som sier om formatering skal være tillatt eller ikke. Tenker at den er grei å ha med for å få samme metodesignatur på alle.

@aulonm
Copy link
Contributor Author

aulonm commented Apr 10, 2025

Vil vi har formatering? tillate feks 12345 67 8903 eller 12345.67.8903

Hvis du ser på de andre metodene har det optiona options parameter som sier om formatering skal være tillatt eller ikke. Tenker at den er grei å ha med for å få samme metodesignatur på alle.

done!

@aulonm aulonm merged commit bb45e14 into main Apr 10, 2025
1 check passed
@aulonm aulonm deleted the norwegian-account-number branch April 10, 2025 08:25
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