Skip to content

Commit dea4013

Browse files
committed
fix(MUSD-431): surface money_home_on as CashSection disable reason
Logger reason ternary previously only knew about flag_off and geo_ineligible, so when isMoneyHomeEnabled (the new gate) was the actual cause of CashSection returning null, the log misreported it as geo_ineligible. Add an explicit money_home_on branch and include the moneyHome flag value in the log line.
1 parent 71753bd commit dea4013

1 file changed

Lines changed: 5 additions & 1 deletion

File tree

app/components/Views/Homepage/Sections/Cash/CashSection.tsx

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,8 +76,12 @@ const CashSection = forwardRef<SectionRefreshHandle, CashSectionProps>(
7676
useImperativeHandle(ref, () => ({ refresh }), [refresh]);
7777

7878
if (!isCashSectionEnabled) {
79+
let reason = 'flag_off';
80+
if (isMusdConversionEnabled) {
81+
reason = !isGeoEligible ? 'geo_ineligible' : 'money_home_on';
82+
}
7983
Logger.log(
80-
`[CashSection] not rendered flag=${isMusdConversionEnabled} geo=${isGeoEligible} reason=${!isMusdConversionEnabled ? 'flag_off' : 'geo_ineligible'}`,
84+
`[CashSection] not rendered flag=${isMusdConversionEnabled} geo=${isGeoEligible} moneyHome=${isMoneyHomeEnabled} reason=${reason}`,
8185
);
8286
return null;
8387
}

0 commit comments

Comments
 (0)