-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
API logic for placing a SMS notifications order #328
Labels
kind/user-story
Used for issues that describes functionality for our users.
Comments
3 tasks
acn-sbuad
changed the title
Endpoint for ordering an SMS notification
Controller logic for placing a SMS notifications order
Dec 8, 2023
acn-sbuad
changed the title
Controller logic for placing a SMS notifications order
API logic for placing a SMS notifications order
Dec 8, 2023
acn-sbuad
added
the
kind/user-story
Used for issues that describes functionality for our users.
label
Dec 11, 2023
5 tasks
For documentation we should also document which mobile numbers we accept and not. @codemascot maybe you can create a PR covering this part of the documentation. Unchecking the documentation until this has been completed |
khanrn
added a commit
that referenced
this issue
Jan 30, 2024
#384) * RecipientExt Phone number property added * SMS notification order request data model added * SMS notification orders controller added * SMS phone number validation initiated * Phone number validation added to singleton * Changed PhoneNumber to MobileNumber * With rebase phone changed to mobile * Change of phone to mobile * Attribute for SMS address point added * Mobile number validation rules regex fix * Validator test initiated for SMS notifiaction order request * Validatee recipient provided for SMS return true added * Validatee recipient provided for SMS return false added * Validate SMS recipient not defined return false added * Validate send time has local timezone return true added * Validate send time has UTC Now timezone return true added * Validate send time has unspecified timezone return false added * Validate Send Numbwer missing returns false * Type added to the validationResult variable * Test for valid SMS number added * Explicit type added to the variable * Explicit type added to the variable * Type definition for recipients variable fixed * Sender Number removed from must rule * Order mapper helper function added for getting mobile number * Norwegian phone number strict validation added * Tests adapted for Norwegian phone numbet validation * Test refactored for readability purpose * Typo fixed * Both MapToRecipientExt tests merged into one * SMS order mapper moved to OrderMapper class * SMS order mapper tests added * Fixed merge conflicts with main * Integration tests added for the SMS notificaitons API * Sender number changed to Altinn for integration tests * PostTests API base path fixed for integration tests * Revert:Removed the k6 tests added with integration tests * Ternary to if-else to remove code smell * Removed 5 digit pass as LinkMobility need 9 digits * Validation logic changed for SMS mobile numbers * RegEx removed from validation logic * Invalid mobile number error message updated Co-authored-by: Stephanie Buadu <[email protected]> * Arguments fixed for the tests * Tests messgae update --------- Co-authored-by: Stephanie Buadu <[email protected]>
5 tasks
Manual testing completed. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
Create a new endpoint for placing an order for an SMS notification.
Controller name SMSNotificationOrderController
Endpoint: POST notfications/api/v1/orders/sms
Considerations
Fått fra Altinn 2:
Vi har også denne sjekken: ^(([0-9]{5})|([0-9]{8})|(00[0-9]{3,})|(+[0-9]{3,}))$
Den godkjenner
5 siffer
8 siffer
00 fulgt av minst tre siffer
Den ligger også i altinn.config: ValidMobileNumber
Det er denne som brukes ved Varslingsadresse validering
Tasks
Acceptance Criteria
The text was updated successfully, but these errors were encountered: