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

intelligent update flags synchronization #1784

Merged

Conversation

@m-mcgowan
Copy link
Contributor

commented May 16, 2019

Problem

The device previously only sends the value of these flags when they have changed from the default. However, they are reset when the device resets, yet the cloud is not informed of this change.

Solution

Send the state of these flags whenever connecting to the cloud, so the cloud has the latest state. This will increase data usage marginally. This feature will be reworked in a later release to allow synchronization with less data use.

Steps to Test

  • Load the intelligent updates test app to a device
  • View the device in the Console
  • Call the function enableUpdates with value 0. Confirm that updates are disabled.
  • Reset the device
  • The console will automatically detect that updates are now enabled.

References


Completeness

  • User is totes amazing for contributing!
  • Contributor has signed CLA (Info here)
  • Problem and Solution clearly stated
  • Run unit/integration/application tests on device
  • Added documentation
  • Added to CHANGELOG.md after merging (add links to docs and issues)

  • [bugfix] intelligent update flags synchronization #1784

@m-mcgowan m-mcgowan added this to the 1.2.0-rc.1 milestone May 16, 2019

@m-mcgowan m-mcgowan requested a review from technobly May 16, 2019

unconditionally sends the updates enabled and updates forced flags wh…
…en connecting to the cloud (new or existing session.). This ensures the cloud is kept in sync. We will later revise this to only send when the data is out of sync.

@technobly technobly force-pushed the feature/intelligent-firmware-updates-synchronization branch from 181786f to 9251a15 May 16, 2019

@technobly technobly merged commit 6b58883 into develop May 16, 2019

1 check passed

continuous-integration/travis-ci/push The Travis CI build passed
Details

@technobly technobly deleted the feature/intelligent-firmware-updates-synchronization branch May 16, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.