Skip to content

fix(#586): gate advanced features behind advancedMode toggle - #644

Merged
Jaydbrown merged 2 commits into
conduit-protocol:mainfrom
Adeyemiadigun:fix/586-advanced-mode-toggle
Oct 1, 2026
Merged

Jaydbrown merged 2 commits into
conduit-protocol:mainfrom
Adeyemiadigun:fix/586-advanced-mode-toggle

Conversation

@Adeyemiadigun

@Adeyemiadigun Adeyemiadigun commented Sep 26, 2026 •

Copy link
Copy Markdown
Contributor
  • hooks/useSettings: expose advancedMode (default false) + same-tab notifySettingsUpdated() so mounted components re-read on toggle
  • settings page: notify subscribers on save; document what the toggle reveals
  • StreamCard: show raw stroops/s rate in advanced mode
  • stream detail: raw stroop amounts inline + Advanced details card with stream/token/factory contract addresses and raw unix timestamps
  • compare page: raw stroop rate + token contract rows in advanced mode
  • navbar: Compare link only in advanced mode (route stays directly reachable)

What does this PR do?

Type of change

  • Bug fix
  • New feature / page / component
  • Refactor
  • Style / design fix
  • Test coverage
  • Documentation
  • Dependency update

Related issue

Closes #586

Changes

File | Change -- | -- hooks/useSettings.ts | Exposes advancedMode (default false) in PublicSettings; adds SETTINGS_UPDATED_EVENT + notifySettingsUpdated() so mounted components re-read settings on same-tab writes app/settings/page.tsx | Dispatches notifySettingsUpdated() after persisting; adds helper text under the toggle explaining what Advanced Mode reveals components/stream/StreamCard.tsx | Shows raw  stroops/s under the formatted rate when advanced mode is on app/stream/[id]/page.tsx | Raw stroop amounts inline (rate, deposited, withdrawn) + conditional "Advanced details" card with stream/token/factory contract addresses, raw stroops, and unix timestamps app/streams/compare/page.tsx | Conditional "Rate (stroops/s)" and "Token contract" rows in advanced mode components/Navbar.tsx | "Compare" nav link (desktop + mobile) rendered only in advanced mode; route itself stays directly reachable

Checklist

  • npm run typecheck — no errors
  • npm run lint — no warnings
  • npm test — all tests pass
  • npm run build — production build succeeds
  • Tested in browser with Freighter on testnet
  • No hue-named Tailwind colour classes added (black/white/gray only)
  • Loading state handled for any new data-fetching UI
  • Error state handled and displayed inline (no alert())
  • 'use client' added only where genuinely needed
  • CHANGELOG.md updated under [Unreleased]

Screenshots

Before After

Notes for reviewers

Adeyemiadigun and others added 2 commits September 26, 2026 11:37
… toggle

- hooks/useSettings: expose advancedMode (default false) + same-tab
  notifySettingsUpdated() so mounted components re-read on toggle
- settings page: notify subscribers on save; document what the toggle reveals
- StreamCard: show raw stroops/s rate in advanced mode
- stream detail: raw stroop amounts inline + Advanced details card with
  stream/token/factory contract addresses and raw unix timestamps
- compare page: raw stroop rate + token contract rows in advanced mode
- navbar: Compare link only in advanced mode (route stays directly reachable)
@Jaydbrown
Jaydbrown merged commit 96a759a into conduit-protocol:main Oct 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Settings: the advancedMode toggle has no effect anywhere in the app

2 participants