Hot/cold combination of a wallet doesn't work smooth

[Keith-CY]

Here is the case

1. Create a wallet in an offline computer as the cold wallet
2. Export extended public key from the cold wallet
3. Import extended public key in an online computer as the hot wallet
4. Export an unsigned transaction from the hot wallet
5. Import the unsigned transaction in the cold wallet and sign&export
6. Try to import the signed transaction to broadcast in the hot wallet, failed due to the disabled offline sign menu item, because of https://github.com/nervosnetwork/neuron/blob/f2f57087f3865096c7d485b6de40df5ef11d772a/packages/neuron-wallet/src/controllers/app/menu.ts#L354, it's disabled when the wallet is a watching wallet

Workaround: create a transient wallet to enable the offline-sign menu item to accept the signed transaction.

---

We should find a better workflow for the combination of cold/hot wallets.

[Danie0918]

PRD

【Optimization of hot/cold combinations for wallets】

• Requirement Description:The combination of hot/cold in the wallet is not smooth and the workflow needs to be optimized.

• Requirement Contents

1. Offline signing is not being supported to import signed transactions and to close the pop-up window when the signature is complete.
2. Added broadcast transaction:

• Unable to select when offline, support for watch-only account broadcasting.
• The broadcast shows the loading status and jumps to history when finished.

3. Pop-up window alert:

• Import signed transaction files in Offline Signature.
• Import unsigned transaction files in broadcast transaction.

[yanguoyu]

1. [Optimization of hot/cold combinations for wallets] Added broadcast transaction:
   support for watch-only account broadcasting.

I believe broadcast transactions can be for any wallet, but not a watch-only account.

[Danie0918]

1. [Optimization of hot/cold combinations for wallets] Added broadcast transaction:
   support for watch-only account broadcasting.

I believe broadcast transactions can be for any wallet, but not a watch-only account.

In that case, the user would still need to create a separate wallet to do so.Does this meet expectations? @Keith-CY

[yanguoyu]

1. [Optimization of hot/cold combinations for wallets] Added broadcast transaction:
   support for watch-only account broadcasting.

I believe broadcast transactions can be for any wallet, but not a watch-only account.

In that case, the user would still need to create a separate wallet to do so.Does this meet expectations? @Keith-CY

My mistake, I mean works for any wallet, not only limited to work with a watched-only account. @Danie0918

[Danie0918]

1. [Optimization of hot/cold combinations for wallets] Added broadcast transaction:
   support for watch-only account broadcasting.

I believe broadcast transactions can be for any wallet, but not a watch-only account.

In that case, the user would still need to create a separate wallet to do so.Does this meet expectations? @Keith-CY

My mistake, I mean works for any wallet, not only limited to work with a watched-only account. @Danie0918

Okay, this is as expected.

[Keith-CY]

Sorry for replying late.

These two sub-items offline sign and broadcast transaction can be merged into one.

If the wallet is able to sign, show offline sign
If the wallet is unable to sign, show broadcast transaction

[Danie0918]

UI：https://www.figma.com/file/6XNoimRDbFTTNm016rbIdU/Magickbase?type=design&node-id=28438-26852&mode=design&t=AY5lLSGCmoHUwzpQ-0

[devchenyan]

fixed: nervosnetwork/neuron#3000

[silySuper]

verified