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

Add new data policy fields #331

Merged
merged 7 commits into from
Jun 27, 2024
Merged

Add new data policy fields #331

merged 7 commits into from
Jun 27, 2024

Conversation

timcowlishaw
Copy link
Contributor

  • enable_forwarding allows forwarding to be enabled or disabled per device

  • precise_location allows users to choose whether precise (5dp) or approxmite (3dp + 2dp of noise) locations are stored.

On deploying we will need to run migrations, then run rake devices:truncate_and_fuzz_locations in order to truncate the existing locations to 5dp.

@timcowlishaw
Copy link
Contributor Author

Addresses #307

@timcowlishaw
Copy link
Contributor Author

We're not convinced that 3dp + noise is quite as "imprecise" as we were hoping: will possibly send a follow up PR to make imprecise locations a little noisier

- enable_forwarding allows forwarding to be enabled or disabled per
  device

- precise_location allows users to choose whether precise (5dp) or
  approxmite (3dp + 2dp of noise) locations are stored.

On deploying we will need to run migrations, then run rake
devices:truncate_and_fuzz_locations in order to truncate the existing
locations to 5dp.
@timcowlishaw timcowlishaw merged commit dfc6e2b into master Jun 27, 2024
2 checks passed
@timcowlishaw timcowlishaw deleted the data-policy-fields branch June 27, 2024 17:02
Copy link

sentry-io bot commented Jun 27, 2024

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ **ActionView::Template::Error: undefined method data_policy' for #<Device id: 16254, owner_id: 7346, name: "CitzenFACE", descri...** V0::MeController#index` View Issue
  • ‼️ **NameError: undefined local variable or method enable_forwarding' for #<Device id: 17545, owner_id: 6556, na...** app/models/device.rb in forward_readings?` View Issue
  • ‼️ **NameError: undefined local variable or method enable_forwarding' for #<Device id: 13104, owner_id: 7659, na...** app/models/device.rb in forward_readings?` View Issue
  • ‼️ **NoMethodError: undefined method precise_location' for #<Device id: nil, owner_id: 6556, name: "TGH_MAIN_NURSEWS...** V0::DevicesController#create` View Issue

Did you find this useful? React with a 👍 or 👎

@timcowlishaw
Copy link
Contributor Author

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

* ‼️ **ActionView::Template::Error: undefined method `data_policy' for #<Device id: 16254, owner_id: 7346, name: "CitzenFACE", descri...** `V0::MeController#index` [View Issue](https://iaac.sentry.io/issues/5544588186/?referrer=github-pr-bot)

Did you find this useful? React with a 👍 or 👎

(This was just momentary due to the migrations not being run yet)

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