kupos-ui-components-lib 10.4.72 → 10.4.74

package/dist/components/DiscountBanners/DiscountBannerDesktop.js

@@ -147,7 +147,6 @@ class DiscountBannersDesktop extends React.Component {
     }
     render() {
         const { services = [], wowDealData, backgroundImages = [], artIncludesChipPill, accentColors = [], label = DEFAULT_LABEL, currencySign = "$", size = "default", selectedKey, className = "", style, } = this.props;
-        console.log("🚀 ~ DiscountBannersDesktop ~ render ~ services:", services);
         // Compute cards directly — no useMemo
         const cards = deriveDiscountCards(services, wowDealData, currencySign);
         if (!cards.length)

package/dist/ui/DiscountBanners/discountUtils.js

@@ -107,6 +107,18 @@ function addOperatorLogo(card, service) {
         { url, name: service.operator_name || ((_f = service.operator_details) === null || _f === void 0 ? void 0 : _f[2]) },
     ];
 }
+// savings_percent may be missing or sent as a string; fall back to the deal's prices so
+// Remate still sorts by its real discount instead of dropping to 0%
+function getWowDealPercent(wowDealData) {
+    const savings = Number(wowDealData.savings_percent);
+    if (savings > 0)
+        return savings;
+    const original = Number(wowDealData.original_price);
+    const final = Number(wowDealData.final_price);
+    return original > 0 && final >= 0 && final < original
+        ? ((original - final) / original) * 100
+        : 0;
+}
 function toDiscountLabel(discount, currencySign) {
     if (discount.type === "fixed") {
         return `${CommonService.currencyRounded(discount.amount, currencySign)} de dcto`;
@@ -118,13 +130,14 @@ export function deriveDiscountCards(services = [], wowDealData, currencySign = "
     var _a, _b, _c, _d, _e;
     const cards = new Map();
     const wowServices = wowDealData === null || wowDealData === void 0 ? void 0 : wowDealData.services;
+    const wowDealPercent = wowDealData ? getWowDealPercent(wowDealData) : 0;
     const hasWowDeal = !!wowDealData &&
         ((Array.isArray(wowServices) && wowServices.length > 0) ||
-            !!wowDealData.savings_percent);
+            wowDealPercent > 0);
     if (hasWowDeal) {
         const remateDiscount = {
             type: "percentage",
-            amount: Math.round(wowDealData.savings_percent || 0),
+            amount: Math.round(wowDealPercent),
         };
         cards.set(REMATE_KEY, {
             key: REMATE_KEY,
@@ -217,8 +230,9 @@ export function deriveDiscountCards(services = [], wowDealData, currencySign = "
             return a.name.localeCompare(b.name, "es", { sensitivity: "base" });
         });
     }
-    // Highest discount first. Percentages and fixed amounts can't be compared, so percentages
-    // lead. The sort is stable, so on a tie Remate stays ahead and the rest keep service order.
+    // Highest discount first, Remate included — it takes its place by its percent rather than
+    // always leading. Percentages and fixed amounts can't be compared, so percentages lead.
+    // The sort is stable, so on a tie Remate stays ahead and the rest keep service order.
     return Array.from(cards.values()).sort((a, b) => {
         if (a.discountType !== b.discountType) {
             return a.discountType === "percentage" ? -1 : 1;

package/dist/ui/FeaturServiceUiMobile/FeatureServiceUiMobile.js

@@ -91,31 +91,13 @@ const FeatureServiceUiMobile = ({ serviceItem, showTopLabel, colors, isSoldOut,
             selectedSlotService.original_price) *
             100)
         : savingsPercent;
-    // Convert "HH:MM" (24h) → "H:MM AM/PM"
-    const formatTimeAmPm = (time) => {
-        const match = time.match(/(\d{1,2}):(\d{2})/);
-        if (!match)
-            return time;
-        let h = parseInt(match[1], 10);
-        const m = match[2];
-        const period = h >= 12 ? "PM" : "AM";
-        h = h % 12 || 12;
-        return `${h}:${m} ${period}`;
-    };
-    // Convert all HH:MM occurrences in a label string to AM/PM
-    const formatLabelAmPm = (label) => label.replace(/(\d{1,2}):(\d{2})/g, (_, hh, mm) => {
-        let h = parseInt(hh, 10);
-        const period = h >= 12 ? "PM" : "AM";
-        h = h % 12 || 12;
-        return `${h}:${mm} ${period}`;
-    });
-    // The label shown on the dropdown button
-    const departureRange = formatLabelAmPm((activeSlot === null || activeSlot === void 0 ? void 0 : activeSlot.label) || `Entre ${dealWindowFrom} y ${dealWindowTo}`);
-    const currentWindowFrom = (activeSlot === null || activeSlot === void 0 ? void 0 : activeSlot.label)
-        ? activeSlot.label.split(" - ")[0]
+    // The label shown on the dropdown button (24h format, e.g. "Entre 16:00 y 20:00")
+    const departureRange = (activeSlot === null || activeSlot === void 0 ? void 0 : activeSlot.label) || `Entre ${dealWindowFrom} y ${dealWindowTo}`;
+    const currentWindowFrom = activeSlot
+        ? commonService.minutesToTime(activeSlot.start)
         : dealWindowFrom;
-    const currentWindowTo = (activeSlot === null || activeSlot === void 0 ? void 0 : activeSlot.label)
-        ? activeSlot.label.split(" - ")[1]
+    const currentWindowTo = activeSlot
+        ? commonService.minutesToTime(activeSlot.end === 1440 ? 1439 : activeSlot.end)
         : dealWindowTo;
     const isItemExpanded = serviceItem.id === isFeatureDropDownExpand ||
         isFeatureDropDownExpand === true;

package/dist/ui/FeatureServiceUI/FeatureServiceUi.js

@@ -105,11 +105,11 @@ const FeatureServiceUi = ({ serviceItem, showTopLabel, getAnimationIcon, cityOri
     // The label shown on the dropdown button
     const departureRange = (activeSlot === null || activeSlot === void 0 ? void 0 : activeSlot.label) || `${dealWindowFrom} y ${dealWindowTo}`;
     // activeSlot?.label || `Entre ${dealWindowFrom} y ${dealWindowTo}`;
-    const currentWindowFrom = (activeSlot === null || activeSlot === void 0 ? void 0 : activeSlot.label)
-        ? activeSlot.label.split(" - ")[0]
+    const currentWindowFrom = activeSlot
+        ? commonService.minutesToTime(activeSlot.start)
         : dealWindowFrom;
-    const currentWindowTo = (activeSlot === null || activeSlot === void 0 ? void 0 : activeSlot.label)
-        ? activeSlot.label.split(" - ")[1]
+    const currentWindowTo = activeSlot
+        ? commonService.minutesToTime(activeSlot.end === 1440 ? 1439 : activeSlot.end)
         : dealWindowTo;
     if (expandedStateMap[serviceItem.id] === undefined) {
         expandedStateMap[serviceItem.id] = true;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "kupos-ui-components-lib",
-  "version": "10.4.72",
+  "version": "10.4.74",
   "description": "A reusable UI components package",
   "publishConfig": {
     "access": "public"

package/src/components/DiscountBanners/DiscountBannerDesktop.tsx

@@ -197,7 +197,6 @@ class DiscountBannersDesktop extends React.Component<
       className = "",
       style,
     } = this.props;
-      console.log("🚀 ~ DiscountBannersDesktop ~ render ~ services:", services)
 
     // Compute cards directly — no useMemo
     const cards = deriveDiscountCards(services, wowDealData, currencySign);

package/src/ui/DiscountBanners/discountUtils.ts

@@ -148,6 +148,18 @@ function addOperatorLogo(
   ];
 }
 
+// savings_percent may be missing or sent as a string; fall back to the deal's prices so
+// Remate still sorts by its real discount instead of dropping to 0%
+function getWowDealPercent(wowDealData: WowDealData): number {
+  const savings = Number(wowDealData.savings_percent);
+  if (savings > 0) return savings;
+  const original = Number(wowDealData.original_price);
+  const final = Number(wowDealData.final_price);
+  return original > 0 && final >= 0 && final < original
+    ? ((original - final) / original) * 100
+    : 0;
+}
+
 function toDiscountLabel(
   discount: ServiceDiscount,
   currencySign: string,
@@ -167,15 +179,16 @@ export function deriveDiscountCards(
   const cards = new Map<string, DiscountBannerCard>();
 
   const wowServices = wowDealData?.services;
+  const wowDealPercent = wowDealData ? getWowDealPercent(wowDealData) : 0;
   const hasWowDeal =
     !!wowDealData &&
     ((Array.isArray(wowServices) && wowServices.length > 0) ||
-      !!wowDealData.savings_percent);
+      wowDealPercent > 0);
 
   if (hasWowDeal) {
     const remateDiscount: ServiceDiscount = {
       type: "percentage",
-      amount: Math.round(wowDealData!.savings_percent || 0),
+      amount: Math.round(wowDealPercent),
     };
     cards.set(REMATE_KEY, {
       key: REMATE_KEY,
@@ -276,8 +289,9 @@ export function deriveDiscountCards(
     });
   }
 
-  // Highest discount first. Percentages and fixed amounts can't be compared, so percentages
-  // lead. The sort is stable, so on a tie Remate stays ahead and the rest keep service order.
+  // Highest discount first, Remate included — it takes its place by its percent rather than
+  // always leading. Percentages and fixed amounts can't be compared, so percentages lead.
+  // The sort is stable, so on a tie Remate stays ahead and the rest keep service order.
   return Array.from(cards.values()).sort((a, b) => {
     if (a.discountType !== b.discountType) {
       return a.discountType === "percentage" ? -1 : 1;

package/src/ui/FeaturServiceUiMobile/FeatureServiceUiMobile.tsx

@@ -131,36 +131,17 @@ const FeatureServiceUiMobile = ({
         )
       : savingsPercent;
 
-  // Convert "HH:MM" (24h) → "H:MM AM/PM"
-  const formatTimeAmPm = (time: string): string => {
-    const match = time.match(/(\d{1,2}):(\d{2})/);
-    if (!match) return time;
-    let h = parseInt(match[1], 10);
-    const m = match[2];
-    const period = h >= 12 ? "PM" : "AM";
-    h = h % 12 || 12;
-    return `${h}:${m} ${period}`;
-  };
-
-  // Convert all HH:MM occurrences in a label string to AM/PM
-  const formatLabelAmPm = (label: string): string =>
-    label.replace(/(\d{1,2}):(\d{2})/g, (_, hh, mm) => {
-      let h = parseInt(hh, 10);
-      const period = h >= 12 ? "PM" : "AM";
-      h = h % 12 || 12;
-      return `${h}:${mm} ${period}`;
-    });
+  // The label shown on the dropdown button (24h format, e.g. "Entre 16:00 y 20:00")
+  const departureRange =
+    activeSlot?.label || `Entre ${dealWindowFrom} y ${dealWindowTo}`;
 
-  // The label shown on the dropdown button
-  const departureRange = formatLabelAmPm(
-    activeSlot?.label || `Entre ${dealWindowFrom} y ${dealWindowTo}`,
-  );
-
-  const currentWindowFrom = activeSlot?.label
-    ? activeSlot.label.split(" - ")[0]
+  const currentWindowFrom = activeSlot
+    ? commonService.minutesToTime(activeSlot.start)
     : dealWindowFrom;
-  const currentWindowTo = activeSlot?.label
-    ? activeSlot.label.split(" - ")[1]
+  const currentWindowTo = activeSlot
+    ? commonService.minutesToTime(
+        activeSlot.end === 1440 ? 1439 : activeSlot.end,
+      )
     : dealWindowTo;
 
   const isItemExpanded =

package/src/ui/FeatureServiceUI/FeatureServiceUi.tsx

@@ -147,11 +147,13 @@ const FeatureServiceUi = ({
     activeSlot?.label || `${dealWindowFrom} y ${dealWindowTo}`;
   // activeSlot?.label || `Entre ${dealWindowFrom} y ${dealWindowTo}`;
 
-  const currentWindowFrom = activeSlot?.label
-    ? activeSlot.label.split(" - ")[0]
+  const currentWindowFrom = activeSlot
+    ? commonService.minutesToTime(activeSlot.start)
     : dealWindowFrom;
-  const currentWindowTo = activeSlot?.label
-    ? activeSlot.label.split(" - ")[1]
+  const currentWindowTo = activeSlot
+    ? commonService.minutesToTime(
+        activeSlot.end === 1440 ? 1439 : activeSlot.end,
+      )
     : dealWindowTo;
 
   if (expandedStateMap[serviceItem.id] === undefined) {