-
Notifications
You must be signed in to change notification settings - Fork 12
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
Feature branch update for berlinger integration #2421
Feature branch update for berlinger integration #2421
Conversation
…r TemperatureBreachRowType
…_sensor crate Include breach type in config description so that it's unique
Matching breach can have different end timestamp; matching config doesn't depend on is_active
Using updated temperature-sensor crate
…before adding the logs & breaches!
…anch-update-for-berlinger-integration
Bundle size differenceComparing this PR to
|
I've update #2383 from this branch so closing |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was part way through reviewing - came back to it and the PR is closed
@@ -4426,6 +4426,7 @@ export type SensorNode = { | |||
}; | |||
|
|||
export enum SensorNodeType { | |||
Berlinger = 'BERLINGER', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😄 I added this in another PR and then removed it! ( taking the minimalist approach.. don't add code that we aren't using and I didn't need it anymore! )
Glad to see that it's back
Sorry no issue, see description below for reference
I need to add upload functionality as per this issue: #2420, which required merge for feature branch into this PR: #1798.
Wanted to do this separately, to make it easier to review next PR.
This PR is just: #1798 updated from cold chain feature branch.
Had to re-instate temperature config repo and migrations, also update graphql schema.