Skip to content

Number of Events Retained as Configurable Value#44

Open
brott8 wants to merge 3 commits intoniruse:mainfrom
brott8:main
Open

Number of Events Retained as Configurable Value#44
brott8 wants to merge 3 commits intoniruse:mainfrom
brott8:main

Conversation

@brott8
Copy link
Copy Markdown

@brott8 brott8 commented Feb 6, 2026

Made it so that the download_images, alerts_count, and hours_back variables are configurable during integration setup. Currently, alerts_count is hard-coded to 5.

@brott8
Copy link
Copy Markdown
Author

brott8 commented Feb 6, 2026

Note that I also tried to allow the integration to be reconfigured after initial setup, though I currently get a message saying:

Config flow could not be loaded: 500 Internal Server Error Server got itself in trouble

Not sure why, but the pull request still gets the variables as successfully configurable during initial setup.

@niruse
Copy link
Copy Markdown
Owner

niruse commented Feb 6, 2026

Is this version works ?, or need some tests ?

@brott8
Copy link
Copy Markdown
Author

brott8 commented Feb 6, 2026

This version works in my setup. Currently running with 10 events on my end, going for a couple of weeks now.

@niruse niruse self-requested a review March 28, 2026 13:15
@niruse
Copy link
Copy Markdown
Owner

niruse commented Mar 28, 2026

I will merge this changes at evening today

@niruse niruse self-requested a review April 7, 2026 05:23
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 this pull request may close these issues.

2 participants