JupiterBroadcasting / jupiterbroadcasting.com

JupiterBroadcasting.com, hugo-based and community-driven!
https://jupiterbroadcasting.com
99 stars 49 forks source link

people pages - What "username" is meant? #280

Open Strit opened 2 years ago

Strit commented 2 years ago

In the contents/people/*.md pages there's a field simply called username.

There is no indication of what platform the username is for. Is it Mumble, Matrix, IRC?

So maybe we can change the field name into a specific name, like mumble_username?

Strit commented 2 years ago

Looking at the code in show-scraper, it looks like it might just be an entity to link people to shows and not actually a username of any kind.

Am I understanding it correctly?

kbondarev commented 2 years ago

It is just a unique ID of each person, that is also used the URL slug.

I guess slug, person_slug, or GUID would've been a better name for this parameter

kbondarev commented 2 years ago

Speaking of, I noticed that it is on the frontend as "@username" under the full name of the arch guest/host. Probably shouldn't be displayed there like that, as it might make ppl think it's an username on social media (like twitter or Instagram)

Strit commented 2 years ago

Yeah, that's why I got confused when I saw my "username", but I have no social media with that handle.

elreydetoda commented 2 years ago

I definitely think we could change they key name for v2 of the JB.com, but till then we can just remove the @username reference in the guest/host template (IIRC, one of them should just be a symlink to the other after my PR that merged for #110)

ironicbadger commented 2 years ago

What is the output of the scraper for this field?

gerbrent commented 2 years ago

https://raw.githubusercontent.com/JupiterBroadcasting/jupiterbroadcasting.com/main/content/people/alex.md

ironicbadger commented 2 years ago

https://github.com/JupiterBroadcasting/jupiterbroadcasting.com/blob/main/content/people/allanjude.md?plain=1#L12

ironicbadger commented 2 years ago

So looking at this we can probably call it whatever we want.

person_uuid might be adequate?

elreydetoda commented 2 years ago

Ya, the only snag is that if we change it in the scraper then we need to update it every place it's referenced in the main JB site as well. So, that's why I was suggesting the v2 release instead of changing something so vital this close to us trying to do v1 release.