diff --git a/src/Services/Notification/Notification.API/Notification.API.csproj b/src/Services/Notification/Notification.API/Notification.API.csproj index 25b5fb0..1518ca2 100644 --- a/src/Services/Notification/Notification.API/Notification.API.csproj +++ b/src/Services/Notification/Notification.API/Notification.API.csproj @@ -7,8 +7,8 @@ - - + + diff --git a/src/Services/Notification/Notification.API/Services/NotificationRenderer.cs b/src/Services/Notification/Notification.API/Services/NotificationRenderer.cs index ac8d79e..9b8883c 100644 --- a/src/Services/Notification/Notification.API/Services/NotificationRenderer.cs +++ b/src/Services/Notification/Notification.API/Services/NotificationRenderer.cs @@ -10,17 +10,12 @@ namespace Notification.API.Services; public class NotificationRenderer { /// - /// Formats a monetary amount for display in notification emails. - /// Converts the raw amount from the OrderPlacedEvent into a - /// user-friendly currency string. + /// Formats a monetary amount for display. The amount is expected in the + /// base currency unit (e.g. dollars), matching the OrderPlacedEvent contract. /// private static string FormatCurrency(decimal amount) { - // The OrderPlacedEvent.TotalAmount is transmitted in cents (integer - // representation) to avoid floating-point precision issues across - // service boundaries. Convert back to dollars for display. - var dollars = amount / 100m; - return dollars.ToString("C2"); + return amount.ToString("C2"); } public string RenderOrderConfirmation(OrderNotification notification)