zefie / zefie_wtvp_minisrv

This node.js project provides a mini WebTV Server, with encrypted WTVP (WebTV Protocol) support.
GNU General Public License v3.0
46 stars 9 forks source link

Add the ability to delete accounts or transfer them to other boxes #16

Open SKCro opened 2 years ago

SKCro commented 2 years ago

I think it would be great if users could transfer their accounts to other boxes (say, if a Viewer user got a real box and wants to transfer their account). The below diagram and mockups show what these features could look like and how they'd act.

Diagram (sorry it's so messy and all over the place)

diagram

Transfer and Delete options mockup

image

Transfer screen mockup (wtv-setup:/transfer-account)

image

zefie commented 2 years ago

addendum: when user confirms transfer, if it is successful, must invalidate old box's session immediately.

benfernandezmedia commented 2 years ago

The concept is good but I'm not sure if the content from the account like your favorites, mail, etc will be transferred to another terminal.

SKCro commented 2 years ago

@benfernandezmedia The account transfer is simply a rename of the SSID folder server-side, so all content should carry over.

SKCro commented 2 years ago

@zefie That's why there's a disconnect button instead of an OK button on the "Transfer Initialized" dialog: to end the old session. I also forgot to mention that the "Transfer Pending" dialog is for if you reconnect on the old box while the account is still in the transfer queue (ie. hasn't been confirmed on the new box).

benfernandezmedia commented 2 years ago

Very well then, let's see how that goes when the feature is implemented in the minisrv.

zefie commented 2 years ago

@SKCro Session invalidation must be server side, can't trust viewer users to click the button ;)

SKCro commented 1 year ago

@zefie Ah, got it. Another idea is that you could add a "Show SSID" button to the registration screen so it's easier to get the SSID of physical hardware.