Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Generalize external links on user profile #1248

Closed
cellio opened this issue Nov 26, 2023 · 0 comments · Fixed by #1524
Closed

Generalize external links on user profile #1248

cellio opened this issue Nov 26, 2023 · 0 comments · Fixed by #1524
Assignees
Labels
area: backend Changes to server-side code complexity: average Not particularly hard, not particularly trivial. priority: medium type: change request New feature or request

Comments

@cellio
Copy link
Member

cellio commented Nov 26, 2023

meta:288948

Currently, the user profile allows adding three "external" connections: web site, Discord ID, Twitter handle. We should keep Discord until we have a better chat solution, but otherwise, let's generalize this. I see two approaches we could take:

  1. The simpler one to lay out: three (two?) unlabelled links, put whatever you want there. Instead of seeing the link icon and the Twitter bird, you'd see up to three link icons with their links, plus Discord would be there.
  2. The "nicer" one, but requires more layout adjustment: the Mastodon approach of some number of editable pairs, label + value, which you can populate however you want. These won't fit in a single line like the current links do.

I am not proposing any sort of validation of these links a la Mastodon.

I'm proposing three slots because I'd like to encourage GitHub links on our production system, so that would allow you personal web site, GitHub, and some social-media site, plus Discord.

We'll need to add some explanatory text in the "Edit Profile" page for these fields, either way, so we can suggest your personal site, GitHub, or wherever else you can be contacted.

This change should include converting existing Twitter links to use one of the new slots, so no information is lost.

@cellio cellio added area: backend Changes to server-side code type: change request New feature or request priority: medium complexity: unassessed Needs further developer investigation before complexity/feasibility can be determined. labels Nov 26, 2023
@ArtOfCode- ArtOfCode- added complexity: average Not particularly hard, not particularly trivial. and removed complexity: unassessed Needs further developer investigation before complexity/feasibility can be determined. labels Dec 5, 2023
@cellio cellio self-assigned this Jan 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: backend Changes to server-side code complexity: average Not particularly hard, not particularly trivial. priority: medium type: change request New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants