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

Service Check Now - enhancement #3

Open
pradt opened this issue May 12, 2024 · 0 comments
Open

Service Check Now - enhancement #3

pradt opened this issue May 12, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@pradt
Copy link
Owner

pradt commented May 12, 2024

The Check Now feature in the view service runs through the same process as a scheduled data fetch, and hence the data is saved in the db.

Couple of options -

  1. store the data into the database distinguish between scheduled fetch and the ad-hoc fetch.
  2. don't store the data and show the data on the screen for the user

Reason
This doesn't pollute data store, and can additional benefits when debugging - such as showing the endpoint responses - for example the immich endpoints are too large to save the raw responses into the db and erorr.logs are difficult to parse manually.

@pradt pradt added the enhancement New feature or request label May 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant