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

Preparation for ReadInputs3 being optional #114

Draft
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

celsworth
Copy link
Owner

@celsworth celsworth commented Nov 24, 2022

We might use this if we request ReadInputs1 & 2 every 10 seconds, but not 3; this mirrors what the app does.

Therefore we'll need to store just the data we have.

Start off by making database columns NULLable, and allowing ReadInputsAll not to have RI3 data.

Part of #113.

We might use this if we request ReadInputs1 & 2 every 10 seconds, but not 3; this mirrors what the app does.

Therefore we'll need to store just the data we have.

Start off by making database columns NULLable.
We might use this if we request ReadInputs1 & 2 every 10 seconds, but not 3; this mirrors what the app does.

Therefore we'll need to store just the data we have.

Start off by making database columns NULLable.
@celsworth celsworth force-pushed the make-read-inputs-3-optional branch from c569c97 to 36da1b9 Compare December 5, 2022 18:25
@celsworth celsworth force-pushed the master branch 2 times, most recently from d41335a to 6043637 Compare October 17, 2023 12:22
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.

1 participant