+ <%= image_tag "/assets/social_link_icons/#{social_link.parsed[:platform].nil? ? 'other' : social_link.parsed[:platform]}.svg",
+ :alt => social_link.parsed[:platform].nil? ? "other" : social_link.parsed[:platform],
+ :class => "me-2" %>
+ <%= link_to social_link.parsed[:name], social_link.url, :class => "text-body-secondary", :rel => "nofollow me" %>
+
+<% end %>
diff --git a/app/views/users/show.html.erb b/app/views/users/show.html.erb
index d479b1d56da..fa3cd88067f 100644
--- a/app/views/users/show.html.erb
+++ b/app/views/users/show.html.erb
@@ -233,10 +233,15 @@
<% end %>
-
- <%= link_to t(".edit_profile"), edit_profile_path, :class => "btn btn-outline-primary" %>
+
+
+ <%= render "social_links/show",
+ :social_links => @user.social_links %>
+ <% if current_user and @user.id == current_user.id %>
+
+ <%= link_to t(".edit_profile"), edit_profile_path, :class => "btn btn-outline-primary" %>
+
+ <% end %>
-<% end %>
+
<%= @user.description.to_html %>
+
diff --git a/config/locales/en.yml b/config/locales/en.yml
index 2a92fbb63c6..2aefcda7dad 100644
--- a/config/locales/en.yml
+++ b/config/locales/en.yml
@@ -1839,6 +1839,10 @@ en:
show: "Show"
delete: "Delete"
undelete: "Undo delete"
+ social_links:
+ legend: Social Profile Links
+ remove: Remove
+ http_parse_error: Url should start with http:// or https://
update:
success: Profile updated.
failure: Couldn't update profile.
@@ -3179,6 +3183,10 @@ en:
ninth: "9th"
tenth: "10th"
time: "Time"
+ profiles:
+ edit:
+ social_links:
+ remove: Remove
query:
node: Node
way: Way
diff --git a/db/migrate/20241128070750_create_social_links.rb b/db/migrate/20241128070750_create_social_links.rb
new file mode 100644
index 00000000000..d22b1bdbb4f
--- /dev/null
+++ b/db/migrate/20241128070750_create_social_links.rb
@@ -0,0 +1,10 @@
+class CreateSocialLinks < ActiveRecord::Migration[7.2]
+ def change
+ create_table :social_links do |t|
+ t.references :user, :null => false, :foreign_key => true, :index => true
+ t.string :url, :null => false
+
+ t.timestamps
+ end
+ end
+end
diff --git a/db/structure.sql b/db/structure.sql
index 9679e0b9256..d1a8ccf92ec 100644
--- a/db/structure.sql
+++ b/db/structure.sql
@@ -1358,6 +1358,38 @@ CREATE TABLE public.schema_migrations (
);
+--
+-- Name: social_links; Type: TABLE; Schema: public; Owner: -
+--
+
+CREATE TABLE public.social_links (
+ id bigint NOT NULL,
+ user_id bigint NOT NULL,
+ url character varying NOT NULL,
+ created_at timestamp(6) without time zone NOT NULL,
+ updated_at timestamp(6) without time zone NOT NULL
+);
+
+
+--
+-- Name: social_links_id_seq; Type: SEQUENCE; Schema: public; Owner: -
+--
+
+CREATE SEQUENCE public.social_links_id_seq
+ START WITH 1
+ INCREMENT BY 1
+ NO MINVALUE
+ NO MAXVALUE
+ CACHE 1;
+
+
+--
+-- Name: social_links_id_seq; Type: SEQUENCE OWNED BY; Schema: public; Owner: -
+--
+
+ALTER SEQUENCE public.social_links_id_seq OWNED BY public.social_links.id;
+
+
--
-- Name: user_blocks; Type: TABLE; Schema: public; Owner: -
--
@@ -1752,6 +1784,13 @@ ALTER TABLE ONLY public.redactions ALTER COLUMN id SET DEFAULT nextval('public.r
ALTER TABLE ONLY public.reports ALTER COLUMN id SET DEFAULT nextval('public.reports_id_seq'::regclass);
+--
+-- Name: social_links id; Type: DEFAULT; Schema: public; Owner: -
+--
+
+ALTER TABLE ONLY public.social_links ALTER COLUMN id SET DEFAULT nextval('public.social_links_id_seq'::regclass);
+
+
--
-- Name: user_blocks id; Type: DEFAULT; Schema: public; Owner: -
--
@@ -2116,6 +2155,14 @@ ALTER TABLE ONLY public.schema_migrations
ADD CONSTRAINT schema_migrations_pkey PRIMARY KEY (version);
+--
+-- Name: social_links social_links_pkey; Type: CONSTRAINT; Schema: public; Owner: -
+--
+
+ALTER TABLE ONLY public.social_links
+ ADD CONSTRAINT social_links_pkey PRIMARY KEY (id);
+
+
--
-- Name: user_blocks user_blocks_pkey; Type: CONSTRAINT; Schema: public; Owner: -
--
@@ -2607,6 +2654,13 @@ CREATE INDEX index_reports_on_issue_id ON public.reports USING btree (issue_id);
CREATE INDEX index_reports_on_user_id ON public.reports USING btree (user_id);
+--
+-- Name: index_social_links_on_user_id; Type: INDEX; Schema: public; Owner: -
+--
+
+CREATE INDEX index_social_links_on_user_id ON public.social_links USING btree (user_id);
+
+
--
-- Name: index_user_blocks_on_creator_id_and_id; Type: INDEX; Schema: public; Owner: -
--
@@ -3002,6 +3056,14 @@ ALTER TABLE ONLY public.user_mutes
ADD CONSTRAINT fk_rails_591dad3359 FOREIGN KEY (owner_id) REFERENCES public.users(id);
+--
+-- Name: social_links fk_rails_6034fd4f62; Type: FK CONSTRAINT; Schema: public; Owner: -
+--
+
+ALTER TABLE ONLY public.social_links
+ ADD CONSTRAINT fk_rails_6034fd4f62 FOREIGN KEY (user_id) REFERENCES public.users(id);
+
+
--
-- Name: oauth_access_tokens fk_rails_732cb83ab7; Type: FK CONSTRAINT; Schema: public; Owner: -
--
@@ -3397,6 +3459,7 @@ INSERT INTO "schema_migrations" (version) VALUES
('23'),
('22'),
('21'),
+('20241128070750'),
('20241023004427'),
('20241022141247'),
('20240913171951'),
diff --git a/test/controllers/profiles_controller_test.rb b/test/controllers/profiles_controller_test.rb
index f388349ab35..75443efa296 100644
--- a/test/controllers/profiles_controller_test.rb
+++ b/test/controllers/profiles_controller_test.rb
@@ -59,5 +59,14 @@ def test_update
get edit_profile_path
assert_select "form > fieldset > div > div.col-sm-10 > div > input[name=avatar_action][checked]", false
assert_select "form > fieldset > div > div.col-sm-10 > div > div.form-check > input[name=avatar_action][checked]", false
+
+ # Updating social links should work
+ put profile_path, :params => { :user => { :description => user.description, :social_links_attributes => [{ :url => "https://test.com/test" }] } }
+ assert_redirected_to user_path(user)
+ follow_redirect!
+ assert_response :success
+ assert_template :show
+ assert_select ".alert-success", /^Profile updated./
+ assert_select "a", "https://test.com/test"
end
end
diff --git a/test/factories/social_link.rb b/test/factories/social_link.rb
new file mode 100644
index 00000000000..fb6f7f6e1b1
--- /dev/null
+++ b/test/factories/social_link.rb
@@ -0,0 +1,6 @@
+FactoryBot.define do
+ factory :social_link do
+ sequence(:url) { |n| "https://test.com/#{n}" }
+ user
+ end
+end
diff --git a/test/models/social_link_test.rb b/test/models/social_link_test.rb
new file mode 100644
index 00000000000..0cf6064856f
--- /dev/null
+++ b/test/models/social_link_test.rb
@@ -0,0 +1,49 @@
+require "test_helper"
+
+class SocialLinkTest < ActiveSupport::TestCase
+ def test_user_required
+ social_link = create(:social_link)
+
+ assert_predicate social_link, :valid?
+ social_link.user = nil
+ assert_not_predicate social_link, :valid?
+ end
+
+ def test_url_required
+ social_link = create(:social_link)
+
+ assert_predicate social_link, :valid?
+ social_link.url = nil
+ assert_not_predicate social_link, :valid?
+ end
+
+ def test_url_https_valid
+ social_link = create(:social_link)
+
+ assert_predicate social_link, :valid?
+ social_link.url = "test"
+ assert_not_predicate social_link, :valid?
+ end
+
+ def test_parsed_platform
+ social_link = create(:social_link, :url => "https://mastodon.social/@test")
+
+ assert_equal "mastodon", social_link.parsed[:platform]
+ assert_equal "test", social_link.parsed[:name]
+ end
+
+ def test_parsed_platform_custom_name
+ social_link = create(:social_link, :url => "https://discord.com/users/0")
+
+ assert_equal "discord", social_link.parsed[:platform]
+ assert_equal "Discord", social_link.parsed[:name]
+ end
+
+ def test_parsed_platform_other
+ url = "https://test.com/test"
+ social_link = create(:social_link, :url => url)
+
+ assert_nil social_link.parsed[:platform]
+ assert_equal social_link.parsed[:name], url
+ end
+end