Skip to content

Add toggle to hide slopes in alarms#449

Open
crsohr wants to merge 1 commit intoJohanDegraeve:developfrom
open-gluck:feature/show-slope-in-alarms
Open

Add toggle to hide slopes in alarms#449
crsohr wants to merge 1 commit intoJohanDegraeve:developfrom
open-gluck:feature/show-slope-in-alarms

Conversation

@crsohr
Copy link
Copy Markdown

@crsohr crsohr commented Jul 30, 2023

This PR adds a new toggle in the Settings to opt-out of seeing slopes in alarm notifications. This setting is enabled by default (preserving the functionality of showing the slope by default).

Rationale: I am using FL2, and slopes are my main source of anxiety in the official app, as I find they are seldom accurate, and seeing “↓↓” on a “low alarm” doesn't help when it's just the sensor being sloppy and incorrect.

IMG_C9A04D22525A-1

@crsohr crsohr force-pushed the feature/show-slope-in-alarms branch from 3ca9788 to cfc51a5 Compare September 18, 2023 16:07
@callms callms force-pushed the feature/show-slope-in-alarms branch 2 times, most recently from 58f43f6 to 5fbf2ed Compare December 3, 2023 14:46
@callms callms force-pushed the feature/show-slope-in-alarms branch from 5fbf2ed to a6ee96d Compare March 1, 2024 17:30
@callms callms force-pushed the feature/show-slope-in-alarms branch from a6ee96d to af6b976 Compare March 16, 2024 08:10
@paulplant paulplant changed the base branch from master to develop March 24, 2024 09:52
@paulplant
Copy link
Copy Markdown
Collaborator

Hi @crsohr , I was looking at this PR the other day and it's a tricky one to include.

The problem is that the notification should show the trend arrow. Adding yet another option which we could argue is just a "personal preference" for just one person who wants it is a bit difficult and we really try to always avoid PRs that are forcing personal preferences of just one person into the app.

I've been using a Libre 3 for the last 12 days whilst testing the heartbeat function and the L3 (via LibreLinkUp) is shockingly noisy, so I understand why the trend arrow should not be really relied upon in this case, but surely the solution should be to maybe add smoothing to the downloaded values to make them less erratic. @JohanDegraeve , what do you think?

@paulplant paulplant added the question Further information is requested label Mar 24, 2024
@callms callms force-pushed the feature/show-slope-in-alarms branch from af6b976 to 07a23e5 Compare June 8, 2024 07:11
@callms callms force-pushed the feature/show-slope-in-alarms branch from 07a23e5 to 2225d90 Compare September 24, 2024 18:34
@callms callms force-pushed the feature/show-slope-in-alarms branch 3 times, most recently from 2ae5000 to a708180 Compare December 14, 2024 07:48
@callms callms force-pushed the feature/show-slope-in-alarms branch from a708180 to 61df1c2 Compare March 16, 2025 10:41
@callms callms force-pushed the feature/show-slope-in-alarms branch from 61df1c2 to 0d620d9 Compare June 7, 2025 06:08
@callms callms force-pushed the feature/show-slope-in-alarms branch from 0d620d9 to 6cb72b4 Compare September 5, 2025 11:39
@callms callms force-pushed the feature/show-slope-in-alarms branch 2 times, most recently from bc62ed4 to fb4dd41 Compare December 4, 2025 13:08
@callms callms force-pushed the feature/show-slope-in-alarms branch from fb4dd41 to 10c8adb Compare March 4, 2026 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

question Further information is requested

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants