Merge pull request #364 from fablabbcn/precise-location-true-by-default

Precise location should be on by default

db/migrate/20241001080033_change_device_precise_location_default.rb

@@ -0,0 +1,5 @@
+class ChangeDevicePreciseLocationDefault < ActiveRecord::Migration[6.1]
+  def change
+    change_column_default :devices, :precise_location, from: false, to: true
+  end
+end

db/schema.rb

@@ -10,7 +10,8 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 2024_08_12_081108) do
+ActiveRecord::Schema.define(version: 2024_10_01_080033) do
+
   # These are extensions that must be enabled in order to support this database
   enable_extension "adminpack"
   enable_extension "hstore"
@@ -104,7 +105,7 @@
     t.string "hardware_name_override"
     t.string "hardware_version_override"
     t.string "hardware_slug_override"
-    t.boolean "precise_location", default: false, null: false
+    t.boolean "precise_location", default: true, null: false
     t.boolean "enable_forwarding", default: false, null: false
     t.index ["device_token"], name: "index_devices_on_device_token", unique: true
     t.index ["geohash"], name: "index_devices_on_geohash"

spec/models/device_spec.rb

@@ -204,6 +204,16 @@
   end
 
   describe "location truncation and fuzzing" do
+
+    context "by default" do
+      it "saves the precise location" do
+        device = create(:device, latitude: 0.12345, longitude: 10.12345)
+        expect(device.precise_location).to eq(true)
+        expect(device.latitude).to eq(0.12345)
+        expect(device.longitude).to eq(10.12345)
+      end
+    end
+
     context "when the location is changed" do
       context "when the device has precise location set" do
         let(:device) {