Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Better explanation of heartbeat notifications #302

Closed
harlo opened this issue Jun 4, 2018 · 3 comments · Fixed by #367
Closed

Better explanation of heartbeat notifications #302

harlo opened this issue Jun 4, 2018 · 3 comments · Fixed by #367

Comments

@harlo
Copy link
Member

harlo commented Jun 4, 2018

Currently, users may only learn of deactivation or crash via heartbeat notifications. In testing, however, our group didn't quite understand the use case; I would recommend highlighting this feature and why it comes in handy in the wizard.

@n8fr8 n8fr8 added this to the Next Sprint milestone Jul 2, 2018
@deviantollam
Copy link

i will say that the first time i got an SMS of just the heart emoji i wasn't realizing it was the heartbeat. i know that sounds silly, but it still might benefit users to see a small message or explanatory text on the configuration screen that "a simple single-character emoji heart will be sent"

this will also help if someone is using a device or system for monitoring that, for whatever reason, doesn't display the emoji Unicode. they'll just wonder why their device is sending them [X] blank characters, heh

@n8fr8
Copy link
Member

n8fr8 commented Sep 13, 2018

Perhaps the first heartbeat of a session could include an explanation message "Heartbeat activate: You will receive a short notification ever X minutes while Haven is still running"

We can also include a setting that uses <3 instead of Unicode, or allows people to configure the character entirely

@lukeswitz
Copy link
Collaborator

  • User configurable message w/explanatory text to initial message, as @deviantollam and @n8fr8 suggested
  • Appended device battery percentage to the alert

Concatenating strings for translation purposes downstream. Feedback very much welcome, PR on the way. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants