feat(home): remove cash in bottom sheet

[satish-ravi]

Description

Confirmed with product that this is no longer required.

Test plan

Unit tests, CI

Related issues

• Part of ACT-1105

Backwards compatibility

Yes

Network scalability

N/A

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.52%. Comparing base (9d9293c) to head (f688a83).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #5071      +/-   ##
==========================================
- Coverage   85.56%   85.52%   -0.05%     
==========================================
  Files         723      722       -1     
  Lines       29489    29392      -97     
  Branches     5099     5074      -25     
==========================================
- Hits        25232    25137      -95     
+ Misses       4023     4020       -3     
- Partials      234      235       +1     

Files	Coverage Δ
src/app/reducers.ts	22.82% <ø> (ø)
src/app/saga.ts	73.16% <ø> (ø)
src/firebase/firebase.ts	38.49% <ø> (ø)
src/firebase/remoteConfigValuesDefaults.ts	100.00% <ø> (ø)
src/home/WalletHome.tsx	99.06% <100.00%> (+1.33%)	⬆️
src/redux/migrations.ts	97.29% <100.00%> (+<0.01%)	⬆️
src/redux/store.ts	78.68% <ø> (ø)
test/schemas.ts	89.88% <100.00%> (+0.03%)	⬆️
test/values.ts	100.00% <ø> (ø)

... and 2 files with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9d9293c...f688a83. Read the comment docs.

[MuckT]

I think cashInBottomSheetEnabled should also be removed from src/statsig/constants.ts, but otherwise LGTM.

[satish-ravi]

I think cashInBottomSheetEnabled should also be removed from src/statsig/constants.ts, but otherwise LGTM.

I left it as is as it was one of the experiment params, we can clean it up when we remove the experiment.