Skip to content

Conversation

@enderahmetyurt
Copy link

@enderahmetyurt enderahmetyurt commented Aug 23, 2025

Description

Previously, our user profiles supported social links for GitHub, Twitter, and LinkedIn, but Bluesky was missing. This PR adds Bluesky handle support so that users can provide and display their Bluesky information alongside other social accounts.

Changes

Added bluesky field to User model and schema.

Updated forms and serializers to include Bluesky handle.

Updated UI components to display Bluesky link where other social accounts are shown.

Added tests to ensure Bluesky data is stored and rendered correctly.

Why

Bluesky adoption is growing within the developer community. Adding support helps us stay relevant and allows users to showcase their presence on this platform as well.

How has this been tested?

✅ Created a new user and confirmed Bluesky handle can be saved.
✅ Updated an existing user profile and verified the handle is editable.
✅ Checked profile view to ensure Bluesky link is displayed correctly alongside GitHub, Twitter, and LinkedIn.
✅ Verified UI behavior: Bluesky logo shows gray when missing, red when present.

  • Manual testing
  • System tests
  • Unit tests
  • None

Checklist

  • CI pipeline is passing
  • My code follows the conventions of this project
  • I have performed a self-review of my code
  • I have commented on my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (if applicable)
  • I have added seed data to the database (if applicable)

Screenshots/Loom

Screenshot 2025-08-23 at 11 08 45 Screenshot 2025-08-23 at 11 09 00 Screenshot 2025-08-23 at 11 09 06

@LuigiR0jas LuigiR0jas self-requested a review August 28, 2025 01:46
@ibramsterdam
Copy link
Collaborator

Hey @enderahmetyurt! Sorry for my late reply. The past few weeks have been as busy as it was fun (So, I was really busy)!

Your PR looks good at first glance! Right before the conference we merged Hiding social links on user profiles when not set. So it makes sense to also include Bluesky if it fits the UI.
Could you resolve the conflicts on your branch so we can review it?
Could you also provide some images on how the UI would look when all 4 social links are present?

Example:

Screenshot 2025-09-10 at 07 55 03

@enderahmetyurt
Copy link
Author

Hey @enderahmetyurt! Sorry for my late reply. The past few weeks have been as busy as it was fun (So, I was really busy)!

Your PR looks good at first glance! Right before the conference we merged Hiding social links on user profiles when not set. So it makes sense to also include Bluesky if it fits the UI. Could you resolve the conflicts on your branch so we can review it? Could you also provide some images on how the UI would look when all 4 social links are present?

Example:

Screenshot 2025-09-10 at 07 55 03

Hello @ibramsterdam No worries :) I can rebase and resolve the conflicts, and also provide new images. I will work on them, and back.

@enderahmetyurt
Copy link
Author

Hey @ibramsterdam I resolved conflicts and add images when 4 all social images set. If you need anything more, let me know 🙏

Screenshot 2025-09-12 at 19 24 28 Screenshot 2025-09-12 at 19 24 34

@ibramsterdam
Copy link
Collaborator

Hey @ibramsterdam I resolved conflicts and add images when 4 all social images set. If you need anything more, let me know 🙏

@enderahmetyurt It looks great! I want to delegate this to Luigi since I do not have approve rights. Especially with the navigational buttons being hidden if a social link is not set, this looks to be a great addition!

@enderahmetyurt
Copy link
Author

Thank you @ibramsterdam I hope @LuigiR0jas can approve it :)

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.

2 participants