MycroftAI / enclosure-mark1

Replacing the faceplate repo
Apache License 2.0
9 stars 8 forks source link

Confirmations needed #66

Closed penrods closed 7 years ago

penrods commented 8 years ago

There should be a confirmation step before clearing the pairing code.

aatchison commented 7 years ago

Do you think that the confirmation in the core will be enough?

penrods commented 7 years ago

Clarifying: When you pick RESET from the unit menu, it clears the pairing immediately and reboots. I think there should be an "Are you sure you want to disassociate this device from your Mycroft account?" confirmation step they have to go through. Picking RESET instead of REBOOT is way too easy.

aatchison commented 7 years ago

I agree %100.

penrods commented 7 years ago

This was fixed by #78