Skip to content

update: tos#184

Open
BlobMaster41 wants to merge 19 commits intomasterfrom
update/tos
Open

update: tos#184
BlobMaster41 wants to merge 19 commits intomasterfrom
update/tos

Conversation

@BlobMaster41
Copy link

Description

Updated TOS

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Performance improvement
  • Refactoring (no functional changes)
  • Documentation update
  • CI/CD changes
  • Dependencies update

Checklist

Build & Tests

  • npm install completes without errors
  • npm run build:chrome builds successfully
  • Extension loads without errors in browser

Code Quality

  • Code follows the project's coding standards
  • No new TypeScript/ESLint warnings introduced
  • Error handling is appropriate
  • Console logs removed (except for error logging)

Documentation

  • Code comments added for complex logic
  • README updated (if applicable)

Security

  • No sensitive data (keys, credentials) committed
  • No new security vulnerabilities introduced
  • Private keys are never logged or exposed
  • User inputs are properly validated

OPWallet Specific

  • Changes work across all supported browsers (Chrome, Firefox, Brave, Edge, Opera)
  • Wallet state management is handled correctly
  • Transaction signing follows security best practices
  • RPC communication is secure
  • Content script isolation is maintained
  • Background/popup communication is secure

Testing

Browser Testing

  • Chrome
  • Firefox
  • Brave
  • Edge
  • Opera

Screenshots

Related Issues


By submitting this PR, I confirm that my contribution is made under the terms of the project's license.

@BlobMaster41 BlobMaster41 added the enhancement New feature or request label Mar 13, 2026
BlobMaster41 and others added 18 commits March 13, 2026 00:05
Update TOS and domain TOS versions to 1.0.1 and align last-update timestamps to 2026-03-12 across configs. Changes update process.env defaults in vite.config.ts and vite.config.background.ts and adjust constants in src/shared/constant/index.ts (including removal of the DOMAIN_TOS_LAST_UPDATE constant). This ensures build-time env values and runtime constants are consistent for the new TOS release.
Add support for ordered lists and list styling across terms and UI components, and normalize typographic quotes/apostrophes in the Terms text. Changes:
- src/legal-documents/Terms.tsx: add ol to LegalDocStyles, replace the USE RESTRICTIONS <ul> with <ol>, and fix mojibake/typographic quote characters and apostrophes throughout the document.
- src/ui/components/AcceptModals/TermsModal.tsx: add ol style and set listStyleType for ul/ol; include ol in passed styles.
- src/ui/pages/Onboarding/steps/OnboardingTOS.tsx and src/ui/pages/Settings/LegalDocumentsScreen.tsx: add ol style and specify listStyleType for ul/ol to ensure consistent rendering.

These changes ensure ordered lists render correctly and improve text punctuation consistency.
Add the 'ol' tag to the list of allowed/handled elements in the Terms of Service modal so ordered lists render correctly. Changes made in src/ui/components/AcceptModals/TermsModal.tsx.
Delete package-lock.json from the repository and apply changes to package.json. The lockfile removal indicates a change in dependency lock strategy or a need to regenerate the lockfile; run npm install (or the project's preferred package manager) to recreate a lockfile if required.
Introduce WalletHealthPopup component (LowBalancePopup and LowUtxoPopup) with styling, backdrop, animations and actions (OK closes, Optimize navigates to UTXOOptimizeScreen). Integrate into WalletTabScreen: import the popups, add state flags (showLowBalancePopup, showLowUtxoPopup, healthCheckDone), and add a one-time health check effect that runs after balance/address data is loaded. The effect shows the low-balance popup when primary balance < 10,000 sats, or the low-UTXO popup when unspent_utxos_count < 5. Popups are rendered only when higher-priority modals (MLDSA backup reminder or duplication alert) are not active.
Replace multiple state flags and a one-time effect with a derived walletHealthStatus and a single dismissal flag. Removed showLowBalancePopup, showLowUtxoPopup and healthCheckDone state and their effect; introduced healthPopupDismissed (resets on mount) and walletHealthStatus useMemo that computes 'low-balance' or 'low-utxos' from accountBalance/addressSummary. Update popup rendering to rely on walletHealthStatus + healthPopupDismissed and keep existing modal priority checks.
Add esbuild (v0.27.4) as a project dependency in package.json to enable fast bundling/build tooling. The package-lock was updated accordingly.
Introduce a new LowBalanceCsvPopup to prompt users to consolidate unlocked CSV1 funds and provide a direct 'Consolidate Now' action. Refactor WalletHealth popups by extracting shared styles/keyframes (backdropStyle, modalStyle, KEYFRAMES_STYLE), reusing icons (LockOutlined, SwapOutlined), and adding locked-CSV informational notes. Update wallet health logic in WalletTabScreen to detect CSV1 unlocked funds and locked CSV2/3/75 funds (returns typed objects with hasLockedCsvFunds), rename walletHealthStatus to walletHealthCheck, and conditionally render the new CSV popup or the existing low-balance/low-utxos popups. Minor UI adjustments to spacing and animations for a more consistent modal appearance.
Rewrite WalletHealthPopup UI and update wallet health detection in WalletTabScreen. Replace LowBalanceCsvPopup with a more detailed CsvFundsWarningPopup (per-type CSV flags: CSV1/CSV2/CSV3/CSV75) and simplify LowBalancePopup (remove CSV locked-note and consolidate action). Update icons/styles, add ExclamationCircleOutlined, and change warning card layouts and copy (CSV2 staking warning, CSV3/75 dApp limitations, CSV1 consolidation note). Change WalletTabScreen logic to: critical low primary balance, CSV consolidation trigger based on total CSV UTXO count (>5) with per-type flags, then low UTXO count; adjust imports and popup usage accordingly.
Consider CSV1 unlocked funds when detecting a critically low primary balance. Previously any primary balance <10,000 sats would immediately return a 'low-balance' type; now the code also reads accountBalance.csv1_unlocked_amount (falling back to '0') and only flags 'low-balance' if CSV1 unlocked sats do not exceed the 10,000 sats threshold, preventing false low-balance warnings when CSV1 funds cover it.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants