althea-net / rita-dash

User dashbaord for managing Rita
https://github.com/althea-net/rita
Apache License 2.0
8 stars 13 forks source link

Text message low balance notification options #67

Closed jtremback closed 5 years ago

jtremback commented 5 years ago

This used to have options for email, but now it's text:

image

jkilpatr commented 5 years ago

well it works with email as well it should check the exit auth method.

jtremback commented 5 years ago

Also I don't know why Yalda used that weird X checkbox, just use a standard bootstrap checkbox element.

asoltys commented 5 years ago

So this setting only gets applied to the currently selected exit? In that case would we hide or disable this form if an exit hasn't been selected yet? What happens if the user changes exits?

Also, do we really want to capture a phone number again here rather than use the one they registered with? Does that imply that they can enter a different number or email than the one they used to sign up with?

One solution might be to have a single checkbox to turn notifications on or off that works regardless of the auth method and gets communicated to whatever exit they connect to. The text could read something like "When low balance notifications are enabled you will receive a text or email alert from your exit server when your account is running low depending on which registration method was used."

jtremback commented 5 years ago

Yea, sorry this would be intended to change the phone number. When the user sees it, it will already have their number in it.

jtremback commented 5 years ago

We should have a conversation to spec the technical and UX elements of this out more.