Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Jimmy Royer <[email protected]>
  • Loading branch information
sastels and jimleroyer committed May 16, 2024
1 parent 554ef3c commit 91aca6a
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions app/delivery/send_to_providers.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
import os
import re
from datetime import datetime
from typing import Any, Dict, Union
from typing import Any, Dict, Optional
from uuid import UUID

import phonenumbers
Expand Down Expand Up @@ -342,9 +342,9 @@ def update_notification_to_sending(notification, provider):
def provider_to_use(
notification_type: str,
notification_id: UUID,
to: Union[str, None] = None,
to: Optional[str] = None,
international: bool = False,
sender: Union[str, None] = None,
sender: Optional[str] = None,
) -> Any:
"""
Get the provider to use for sending the notification.
Expand Down

0 comments on commit 91aca6a

Please sign in to comment.