remove deprecated avatar and upload code

app/controllers/v0/me_controller.rb

@@ -39,8 +39,6 @@ def user_params
         :country_code,
         :profile_picture,
         :url,
-        :avatar,
-        :avatar_url
       )
     end
 

app/controllers/v0/static_controller.rb

@@ -116,7 +116,6 @@ def search
             h['url'] = v0_device_url(s.searchable_id)
           elsif s.searchable_type == 'User'
             h['username'] = s.searchable.username
-            h['avatar'] = s.searchable.avatar
             h['profile_picture'] = profile_picture_url(s.searchable)
             h['city'] = s.searchable.city
             h['url'] = v0_user_url(s.searchable_id)

app/controllers/v0/users_controller.rb

@@ -61,9 +61,7 @@ def user_params
         :password,
         :city,
         :country_code,
-        :url,
-        :avatar,
-        :avatar_url
+        :url
       )
     end
 

app/models/user.rb

@@ -33,7 +33,7 @@ class User < ActiveRecord::Base
 
   has_many :devices, foreign_key: 'owner_id', after_add: :update_cached_device_ids!, after_remove: :update_cached_device_ids!
   has_many :sensors, through: :devices
-  has_many :uploads, dependent: :destroy
+
   has_many :oauth_applications, class_name: 'Doorkeeper::Application', as: :owner
   has_one_attached :profile_picture
 
@@ -44,7 +44,7 @@ def self.ransackable_attributes(auth_object = nil)
   end
 
   def self.ransackable_associations(auth_object = nil)
-    ["devices", "sensors", "uploads"]
+    ["devices", "sensors"]
   end
 
   def archive
@@ -63,10 +63,6 @@ def access_token
     Doorkeeper::AccessToken.find_or_initialize_by(application_id: 4, resource_owner_id: id)
   end
 
-  def avatar
-    avatar_url || "https://smartcitizen.s3.amazonaws.com/avatars/default.svg"
-  end
-
   def access_token!
     access_token.expires_in = 2.days.from_now
     access_token.save
@@ -136,14 +132,6 @@ def update_all_device_ids!
     update_column(:cached_device_ids, device_ids.try(:sort))
   end
 
-  def self.check_bad_avatar_urls
-    User.where.not(avatar_url: nil).each do |user|
-      unless `curl -I #{user.avatar_url} 2>/dev/null | head -n 1`.split(' ')[1] == "200"
-        puts [user.id, user.avatar_url].join(' - ')
-      end
-    end
-  end
-
 private
 
   def check_if_users_have_valid_email

app/views/v0/devices/_device.jbuilder

@@ -33,7 +33,6 @@ if with_owner && device.owner
     json.id device.owner.id
     json.uuid device.owner.uuid
     json.username device.owner.username
-    json.avatar device.owner.avatar
 
     json.profile_picture profile_picture_url(device.owner)
 

app/views/v0/users/_user.jbuilder

@@ -3,7 +3,6 @@ json.(user,
   :uuid,
   :role,
   :username,
-  :avatar,
   :profile_picture,
   :url,
   :location,

config/routes.rb

@@ -26,10 +26,6 @@
     resources :components, only: [:show, :index]
     resources :sessions, only: :create
 
-    resources :uploads, path: 'avatars' do
-      post 'uploaded' => 'uploads#uploaded', on: :collection
-    end
-
     resources :tag_sensors
     resources :tags
     resources :measurements
@@ -38,8 +34,6 @@
     resources :oauth_applications, path: 'applications'
 
     resources :me, only: [:index] do
-      patch 'avatar' => 'uploads#create', on: :collection
-      post 'avatar' => 'uploads#create', on: :collection
       patch '/' => 'me#update', on: :collection
       put '/' => 'me#update', on: :collection
       delete '/' => 'me#destroy', on: :collection

db/migrate/20231221142429_remove_avatars_and_uploads.rb

@@ -0,0 +1,6 @@
+class RemoveAvatarsAndUploads < ActiveRecord::Migration[6.1]
+  def change
+    drop_table :uploads
+    remove_column :users, :avatar_url
+  end
+end

db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 2023_10_06_064514) do
+ActiveRecord::Schema.define(version: 2023_12_21_142429) do
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "adminpack"
@@ -261,19 +261,6 @@
     t.index ["name"], name: "index_tags_on_name", unique: true
   end
 
-  create_table "uploads", id: :serial, force: :cascade do |t|
-    t.string "type"
-    t.string "original_filename"
-    t.jsonb "metadata"
-    t.datetime "created_at", null: false
-    t.datetime "updated_at", null: false
-    t.uuid "uuid", default: -> { "uuid_generate_v4()" }
-    t.integer "user_id"
-    t.string "key"
-    t.index ["type"], name: "index_uploads_on_type"
-    t.index ["user_id"], name: "index_uploads_on_user_id"
-  end
-
   create_table "users", id: :serial, force: :cascade do |t|
     t.string "username"
     t.string "email"
@@ -284,7 +271,6 @@
     t.string "city"
     t.string "country_code"
     t.string "url"
-    t.string "avatar_url"
     t.integer "role_mask", default: 0, null: false
     t.uuid "uuid", default: -> { "uuid_generate_v4()" }
     t.string "legacy_api_key", null: false
@@ -304,5 +290,4 @@
   add_foreign_key "devices_tags", "tags"
   add_foreign_key "postprocessings", "devices"
   add_foreign_key "sensors", "measurements"
-  add_foreign_key "uploads", "users"
 end

spec/controllers/v0/me_controller_spec.rb

@@ -1,5 +1,5 @@
 require 'rails_helper'
 
 RSpec.describe V0::MeController do
-  skip { is_expected.to permit(:email,:username,:password,:city,:country_code,:url,:avatar,:avatar_url).for(:update) }
+  skip { is_expected.to permit(:email,:username,:password,:city,:country_code,:url).for(:update) }
 end

spec/controllers/v0/users_controller_spec.rb

@@ -1,5 +1,5 @@
 require 'rails_helper'
 
 RSpec.describe V0::UsersController do
-  it { is_expected.to permit(:email,:username,:password,:city,:country_code,:url,:avatar,:avatar_url).for(:create) }
+  it { is_expected.to permit(:email,:username,:password,:city,:country_code,:url).for(:create) }
 end