Trasnform site-specific social media columns into bio markdown

Author: pcantrell

app/controllers/participants_controller.rb

@@ -93,8 +93,8 @@ def confirm_email
   def participant_params
     params.require(controller_name.singularize).permit(
       :name, :email, :password,
-      :bio, :github_profile_username,
-      :twitter_handle, :code_of_conduct_agreement,
+      :bio,
+      :code_of_conduct_agreement,
       :contact_details
     )
   end

app/models/participant.rb

@@ -81,10 +81,6 @@ def attending_session?(session)
     sessions_attending.include?(session)
   end
 
-  def github_profile_url
-    "https://github.com/#{self.github_profile_username}"
-  end
-
   def self.find_by_case_insensitive_email(email)
     where(['lower(email) = ?', email.to_s.downcase]).first
   end

app/models/sessions_json_builder.rb

@@ -5,9 +5,6 @@ def to_hash(session)
     { id: s.id,
       participant_id: s.participant.id,
       presenter_name: s.participant.name,
-      presenter_twitter_handle: s.participant.twitter_handle,
-      presenter_github_username: s.participant.github_profile_username,
-      presenter_github_og_image: s.participant.github_og_image,
       presenter_bio: s.participant.bio,
       session_title: s.title,
       summary: s.summary,

app/views/participants/edit.html.erb

@@ -6,8 +6,6 @@
   <%= f.inputs do %>
     <%= f.input :name, :label => 'Your name', :hint => "Please use your real name. This will be used in our printed materials." %>
     <%= f.input :email, :label => 'Your email', :hint => "Please use a real email address. We need this to contact you about your presentation." %>
-    <%= f.input :github_profile_username, :label => 'Your GitHub username' %>
-    <%= f.input :twitter_handle, :label => 'Your Twitter handle' %>
     <%= f.input :bio, :hint => 'You can use <a href="http://daringfireball.net/projects/markdown/syntax">Markdown</a> syntax here. Examples: <strong>**bold**</strong>, <em>*italic*</em>, [link](http://example.com)'.html_safe %>
     <%=
       f.input :code_of_conduct_agreement,

app/views/participants/show.html.erb

@@ -6,22 +6,6 @@
   <% end %>
 <% end %>
 
-<div class="row">
-  <% if @participant.github_profile_username.present? %>
-    <div class="grid_2 column">
-      <h4>GitHub:</h4>
-      <div><%= link_to @participant.github_profile_username, @participant.github_profile_url, target: "blank" %></div>
-    </div>
-  <% end %>
-
-  <% if @participant.twitter_handle.present? %>
-    <div class="grid_3 column">
-      <h4>Twitter:</h4>
-      <div>@<%= @participant.twitter_handle %></div>
-    </div>
-  <% end %>
-</div>
-
 <div class="row bio">
   <h4>Bio</h4>
 

db/migrate/20250309183552_switch_to_freeform_social_links.rb

@@ -0,0 +1,55 @@
+class SwitchToFreeformSocialLinks < ActiveRecord::Migration[7.1]
+  def up
+    execute("select * from participants").each do |result|
+      # Replace existing site-specific social media columns with bio markdown
+      # that mimics the previous formatting of the two supported sites
+
+      social_items = []
+      if slurped = result["github_profile_username"].presence
+        social_items << "[GitHub](https://github.com/#{extract_username(slurped)})"
+      end
+      if cursed = result["twitter_handle"].presence
+        social_items << "Twitter: @#{extract_username(cursed)}"
+      end
+
+      if social_items.any?
+        bio = result["bio"]
+
+        bio.gsub!(/\s+\Z/, '')
+        bio << "\n\n**Links:**\n\n"
+
+        bio << social_items.map do |item|
+          "- #{item}"
+        end.join("\n")
+
+        # Data-updating Rails migrations must use raw SQL instead of models to
+        # ensure migrations continue to work even if models change
+
+        update(
+          "update participants set bio = $1 where id = $2",
+          "Update social links in bios",
+          [bio, result["id"]]
+        )
+      end
+    end
+
+    remove_column :participants, :github_profile_username
+    remove_column :participants, :github_og_url   # Seems to be 98% redundant with prev col; ignoring
+    remove_column :participants, :github_og_image # Unused...except in REST API, which is itself probably unused
+    remove_column :participants, :twitter_handle
+  end
+
+  def down
+    raise "Migration not reversible, because it would require parsing markdown to recover links"
+  end
+
+private
+  
+  def extract_username(username_or_url)
+    if username_or_url =~ %r{https?://(?:.*)/(\w+)}
+      $1  # heck with it, last segment of URL path is probably a username…right?
+    else
+      username_or_url
+    end
+  end
+end

db/schema.rb

@@ -2,15 +2,16 @@
 # of editing this file, please use the migrations feature of Active Record to
 # incrementally modify your database, and then regenerate this schema definition.
 #
-# Note that this schema.rb definition is the authoritative source for your
-# database schema. If you need to create the application database on another
-# system, you should be using db:schema:load, not running all the migrations
-# from scratch. The latter is a flawed and unsustainable approach (the more migrations
-# you'll amass, the slower it'll run and the greater likelihood for issues).
+# This file is the source Rails uses to define your schema when running `bin/rails
+# db:schema:load`. When creating a new database, `bin/rails db:schema:load` tends to
+# be faster and is potentially less error prone than running all of your
+# migrations from scratch. Old migrations may fail to apply correctly if those
+# migrations use external dependencies or application code.
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 2025_02_09_180413) do
+ActiveRecord::Schema[7.1].define(version: 2025_03_09_183552) do
+  create_schema "heroku_ext"
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "pg_stat_statements"
@@ -19,8 +20,8 @@
   create_table "attendances", id: :serial, force: :cascade do |t|
     t.integer "session_id", null: false
     t.integer "participant_id", null: false
-    t.datetime "created_at", null: false
-    t.datetime "updated_at", null: false
+    t.datetime "created_at", precision: nil, null: false
+    t.datetime "updated_at", precision: nil, null: false
     t.index ["session_id", "participant_id"], name: "index_attendances_on_session_id_and_participant_id", unique: true
   end
 
@@ -32,25 +33,25 @@
   create_table "categorizations", id: :serial, force: :cascade do |t|
     t.integer "category_id", null: false
     t.integer "session_id", null: false
-    t.datetime "created_at", null: false
-    t.datetime "updated_at", null: false
+    t.datetime "created_at", precision: nil, null: false
+    t.datetime "updated_at", precision: nil, null: false
     t.index ["category_id", "session_id"], name: "index_categorizations_on_category_id_and_session_id", unique: true
   end
 
   create_table "code_of_conduct_agreements", force: :cascade do |t|
     t.bigint "participant_id", null: false
     t.bigint "event_id", null: false
-    t.datetime "created_at", null: false
-    t.datetime "updated_at", null: false
+    t.datetime "created_at", precision: nil, null: false
+    t.datetime "updated_at", precision: nil, null: false
     t.index ["event_id"], name: "index_code_of_conduct_agreements_on_event_id"
     t.index ["participant_id"], name: "index_code_of_conduct_agreements_on_participant_id"
   end
 
   create_table "events", id: :serial, force: :cascade do |t|
     t.string "name", limit: 255, null: false
     t.date "date", null: false
-    t.datetime "created_at", null: false
-    t.datetime "updated_at", null: false
+    t.datetime "created_at", precision: nil, null: false
+    t.datetime "updated_at", precision: nil, null: false
   end
 
   create_table "levels", id: :serial, force: :cascade do |t|
@@ -61,51 +62,47 @@
     t.string "name", null: false
     t.string "slug", null: false
     t.string "markdown", null: false
-    t.datetime "created_at", null: false
-    t.datetime "updated_at", null: false
+    t.datetime "created_at", precision: nil, null: false
+    t.datetime "updated_at", precision: nil, null: false
     t.index ["slug"], name: "index_markdown_contents_on_slug", unique: true
   end
 
   create_table "participants", id: :serial, force: :cascade do |t|
     t.string "name", limit: 255
     t.string "email", limit: 255
     t.text "bio"
-    t.datetime "created_at", null: false
-    t.datetime "updated_at", null: false
+    t.datetime "created_at", precision: nil, null: false
+    t.datetime "updated_at", precision: nil, null: false
     t.string "crypted_password", limit: 255
     t.string "persistence_token", limit: 255
     t.string "perishable_token", limit: 255, default: "", null: false
-    t.string "github_profile_username"
-    t.string "github_og_image"
-    t.string "github_og_url"
-    t.string "twitter_handle"
-    t.datetime "email_confirmed_at"
+    t.datetime "email_confirmed_at", precision: nil
     t.index ["email"], name: "index_participants_on_email", unique: true
     t.index ["perishable_token"], name: "index_participants_on_perishable_token"
   end
 
   create_table "presentations", id: :serial, force: :cascade do |t|
     t.integer "session_id"
     t.integer "participant_id"
-    t.datetime "created_at", null: false
-    t.datetime "updated_at", null: false
+    t.datetime "created_at", precision: nil, null: false
+    t.datetime "updated_at", precision: nil, null: false
   end
 
   create_table "presenter_timeslot_restrictions", id: :serial, force: :cascade do |t|
     t.integer "participant_id"
     t.integer "timeslot_id"
     t.float "weight"
-    t.datetime "created_at", null: false
-    t.datetime "updated_at", null: false
+    t.datetime "created_at", precision: nil, null: false
+    t.datetime "updated_at", precision: nil, null: false
     t.index ["timeslot_id", "participant_id"], name: "present_timeslot_participant_unique", unique: true
   end
 
   create_table "rooms", id: :serial, force: :cascade do |t|
     t.integer "event_id", null: false
     t.string "name", limit: 255, null: false
     t.integer "capacity"
-    t.datetime "created_at", null: false
-    t.datetime "updated_at", null: false
+    t.datetime "created_at", precision: nil, null: false
+    t.datetime "updated_at", precision: nil, null: false
     t.boolean "schedulable", default: true
   end
 
@@ -115,8 +112,8 @@
     t.text "description", null: false
     t.boolean "panel", default: false, null: false
     t.boolean "projector", default: false, null: false
-    t.datetime "created_at", null: false
-    t.datetime "updated_at", null: false
+    t.datetime "created_at", precision: nil, null: false
+    t.datetime "updated_at", precision: nil, null: false
     t.integer "event_id"
     t.integer "timeslot_id"
     t.integer "room_id"
@@ -136,10 +133,10 @@
 
   create_table "timeslots", id: :serial, force: :cascade do |t|
     t.integer "event_id", null: false
-    t.datetime "created_at", null: false
-    t.datetime "updated_at", null: false
-    t.datetime "starts_at"
-    t.datetime "ends_at"
+    t.datetime "created_at", precision: nil, null: false
+    t.datetime "updated_at", precision: nil, null: false
+    t.datetime "starts_at", precision: nil
+    t.datetime "ends_at", precision: nil
     t.boolean "schedulable", default: true
     t.string "title"
   end

lib/tasks/app.rake

@@ -621,7 +621,6 @@ namespace :app do
       participant.email = FFaker::Internet.safe_email
       participant.password = 'standard'
       participant.bio = FFaker::Lorem.paragraph if [true, false].sample
-      participant.twitter_handle = FFaker::Internet.user_name(participant.name) if [true, false].sample
       participant.save!
       progress.increment
     end

spec/controllers/participants_controller_spec.rb

@@ -102,13 +102,11 @@
 
       describe "more attributes are not required" do
         it "should be successful" do
-          put :update, params: {id: joe, participant: {
-                                twitter_handle: 'schmoe',
-                                github_profile_username: 'jschmoe'}
-                               }
+          put :update, params: {
+            id: joe, participant: { bio: 'schmoe' }
+          }
           expect(response).to be_redirect
-          expect(joe.reload.twitter_handle).to eq 'schmoe'
-          expect(joe.github_profile_username).to eq 'jschmoe'
+          expect(joe.reload.bio).to eq 'schmoe'
         end
       end
 

spec/factories/participant.rb

@@ -16,10 +16,6 @@
     factory :luke do
       email { "[email protected]" }
       name { "Luke Francl" }
-      github_profile_username { "look" }
-      github_og_image { "https://avatars1.githubusercontent.com/u/10186?v=3&s=400" }
-      github_og_url { "https://github.com/look" }
-      twitter_handle { "lof" }
       bio { "the man with the master plan" }
     end
   end

spec/features/manage_your_profile_spec.rb

@@ -12,14 +12,10 @@
 
     scenario "I can update my profile attributes" do
       click_link "Welcome Joe Schmoe"
-      fill_in 'Your GitHub username', with: "JoeSchmoeGithubUsername"
-      fill_in 'Your Twitter handle', with: "JoeSchmoeTwitterHandle"
       bio = FFaker::HipsterIpsum.paragraph(3)
       fill_in 'Bio',  with: bio
       click_button "Update Profile"
 
-      expect(page).to have_content "JoeSchmoeGithubUsername"
-      expect(page).to have_content "JoeSchmoeTwitterHandle"
       expect(page).to have_content bio
     end
   end

spec/models/participant_spec.rb

@@ -1,15 +1,6 @@
 require 'spec_helper'
 
 describe Participant do
-
-  describe "github profile url" do
-    let(:luke)  { create(:luke) }
-
-    it "can be constructed with a github username" do
-      expect(luke.github_profile_url).to eq "https://github.com/look"
-    end
-  end
-
   describe "timeslot restrictions" do
     let(:event) { create(:event) }
     let!(:time1) { create(:timeslot_1, event: event) }

spec/models/sessions_json_builder_spec.rb

@@ -13,9 +13,6 @@
       expect(h[:participant_id]).to be session.participant_id
 
       expect(h[:presenter_name]).to be session.participant.name
-      expect(h[:presenter_twitter_handle]).to be session.participant.twitter_handle
-      expect(h[:presenter_github_username]).to be session.participant.github_profile_username
-      expect(h[:presenter_github_og_image]).to be session.participant.github_og_image
       expect(session.participant.bio).to_not be_nil
       expect(h[:presenter_bio]).to be session.participant.bio
       expect(h[:session_title]).to be session.title