search

meower-media / server

Official source code for the backend that powers the Meower social media platform.
MIT License
12 stars 15 forks source link

Feat: Pronouns #274

Closed showierdata9978 closed 3 months ago

showierdata9978 commented 3 months ago

closes #272

esk33 commented 3 months ago

Nooo they're making Meower woke 🤬🤬🤬

showierdata9978 commented 3 months ago

Nooo they're making Meower woke 🤬🤬🤬

this is a joke right?

JoshAtticus commented 3 months ago

Nooo they're making Meower woke 🤬🤬🤬

how could they this is unacceptable we need to boycott meower and use facebook instead 🤬🤬🤬🤬🤬🤬🤬🤬🤬

JoshAtticus commented 3 months ago

but being fr, having an array and stuff is too complex, it can just be a single string, like tnix also said on meower, see https://leo.atticat.tech/share/?id=4bff790b-cca9-42e5-834f-99b0868f9680

esk33 commented 3 months ago

I agree with Josh, this seems like an overengineered solution considering the problem it is trying to solve

showierdata9978 commented 3 months ago

I ain't changing the format

JoshAtticus commented 3 months ago

I ain't changing the format

then this pr should be closed

mybearworld commented 3 months ago

It might be a bit difficult to create a UI for entering these like that with clients. I think just using strings would be better.

{
  "pronouns": ["he/him", "they/them"]
}
tnix100 commented 3 months ago

It might be a bit difficult to create a UI for entering these like that with clients. I think just using strings would be better.

{
  "pronouns": ["he/him", "they/them"]
}

I think this is what's going to be implemented now.

showierdata9978 commented 3 months ago

yep

CST1229 commented 3 months ago

I would suggest going the Discord route and just having one text field.

{
"pronouns": "he/they"
}
showierdata9978 commented 3 months ago

that was already 100% ruled out.

JoshAtticus commented 3 months ago

that was already 100% ruled out.

you don't make that decision afaik

showierdata9978 commented 3 months ago

image

JoshAtticus commented 3 months ago

image

who said it was going to be merged then?

showierdata9978 commented 3 months ago

This should be changed to just a string. I still think an array of strings is too much.

no

mybearworld commented 3 months ago

This should be changed to just a string. I still think an array of strings is too much.

Why is it too much? It would be nice to be able to have a "preferred" pronoun that could be displayed inline on posts, but to also have another/others that could be displayed on a profile, for instance

tnix100 commented 3 months ago

This should be changed to just a string. I still think an array of strings is too much.

Why is it too much? It would be nice to be able to have a "preferred" pronoun that could be displayed inline on posts, but to also have another/others that could be displayed on a profile, for instance

You can always use your quote to have more, or link to something like pronouns.page. I don't think Meower as a chat platform needs to have all your pronouns, just the one you most prefer is enough in my opinion.

mybearworld commented 3 months ago

You can always use your quote to have more, or link to something like pronouns.page. I don't think Meower as a chat platform needs to have all your pronouns, just the one you most prefer is enough in my opinion.

That makes sense

williamhorning commented 3 months ago

pronoun support is something i'm in favor of adding but the amount of bikeshedding and the focus on rewriting events makes this extremely low priority