beeminder / beedroid

This is a repository to keep track of issues with the Beeminder Android app
6 stars 0 forks source link

Ruthlessly purge settings: Zeno enabled and Zeno Start et al in Goal settings #62

Closed dreeves closed 1 year ago

dreeves commented 5 years ago

image

Notification settings

  1. panic buffer (number of days)
  2. zeno start time
  3. zeno checkbox

It's decided that we're killing panic buffer. Zeno checkbox and start time stay, until later when we use beeminder.com/reminders on the website for setting all reminders including Beedroid.

Title

This dies because it's settable on the website (not to mention the turmoil with slugs/goalnames/blurbs/descriptions/titles/fineprint/ids that we're still sorting out).

Keep it secret. Keep it safe.

Dies (settable on website).

Make datapoints public.

Dies (settable on website).

dreeves commented 5 years ago

Update: we can kill Goal Settings completely since we have the fwomp page of zeno start time and zeno checkboxes for each goal in general settings under "Emergency day notifications"!

adamwolf commented 5 years ago

Panic buffer days is used in the app as the number of days before deadline to show Zeno reminders. This should pull from leadtime for the goal on the website, right @dreeves?

adamwolf commented 5 years ago

Can we not mark this Mendoza for the next release? I'm skeptical we can do this change and get it working well before our May 28th deadline, because of something I just realized.

We actually don't have a built-in way right now for notifying in a non-zeno way. i.e. if you have lead-days set to 7, you're going to be on the zeno schedule for all of those 7 days, rather than "once daily until emergency, then zeno".

I do not think this is a big change, and if it was the only thing on the list for this week I think it would be fine, but combined with the other things, I'd rather leave it like this until the next release. There's also a chance that we'll get some emergency user feedback from this release and I want to reserve some time for that too.

dreeves commented 5 years ago

Agreed. I don't know why I mendoza'd it in the first place. We should only mendoza things we have to (tentatively including all regressions per the pareto dominance principle but we can debate those)!

adamwolf commented 4 years ago

This was done in f2d8711d792d6674236329d1576e2839139c2ce2, which is already in the 3.3.x betas, but not yet released to the public.

adamwolf commented 3 years ago

This has been released.