Skip to content

Commit ede8d00

Browse files
authored
Merge pull request #6069 from EdgeApp/jon/hide-raw-keys-dev-mode
Hide Get Raw Keys behind Developer Mode
2 parents c12a7bb + c35e3d6 commit ede8d00

2 files changed

Lines changed: 9 additions & 0 deletions

File tree

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
## Unreleased (develop)
44

55
- changed: Route maestro test builds to a dedicated Zealot channel so they no longer appear in the production release list.
6+
- changed: Hide the wallet "Get Raw Keys" option behind Developer Mode, while still showing it for wallets that fail to load.
67

78
## 4.49.0 (staging)
89

src/components/modals/WalletListMenuModal.tsx

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -147,6 +147,9 @@ export const WalletListMenuModal: React.FC<Props> = props => {
147147
const pausedWallets = useSelector(
148148
state => state.ui.settings.userPausedWalletsSet
149149
)
150+
const developerModeOn = useSelector(
151+
state => state.ui.settings.developerModeOn
152+
)
150153

151154
const wallet = useWatch(account, 'currencyWallets')[walletId]
152155

@@ -274,6 +277,11 @@ export const WalletListMenuModal: React.FC<Props> = props => {
274277
)
275278
continue
276279

280+
// Hide `getRawKeys` behind Developer Mode. Wallets that fail to load
281+
// still expose raw keys via the `wallet == null` branch above, so a
282+
// broken wallet can always be recovered regardless of this setting.
283+
if (value === 'getRawKeys' && !developerModeOn) continue
284+
277285
result.push({ label, value })
278286
}
279287

0 commit comments

Comments
 (0)