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

feat(profile info): add custom genders #718

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

n1rwana
Copy link
Member

@n1rwana n1rwana commented Sep 8, 2022

image
image

Resolves #312

@vlapskii
Copy link
Contributor

vlapskii commented Sep 8, 2022

Не вижу смысла в этом нововведении.

@n1rwana
Copy link
Member Author

n1rwana commented Sep 8, 2022

Я лишь выполнил Issue #312 ¯\_(ツ)_/¯

@vlapskii
Copy link
Contributor

vlapskii commented Sep 8, 2022

Я лишь выполнил Issue #312 ¯\_(ツ)_/¯

Это конечно хорошо, но оригинальный вк даже не предлагает другой пол кроме муж/жен.

@ByloTonix
Copy link

@n1rwana Лишнее

@n1rwana
Copy link
Member Author

n1rwana commented Sep 8, 2022

Ок.

@n1rwana n1rwana closed this Sep 8, 2022
@n1rwana n1rwana deleted the other-gender-options branch September 8, 2022 13:37
@ByloTonix
Copy link

@n1rwana Сделай лучше музыку)))))))

@SuperErnD
Copy link
Contributor

@n1rwana Сделай лучше музыку)))))))

согл

@SuperErnD
Copy link
Contributor

за голоса))

@ghost
Copy link

ghost commented Sep 8, 2022

Идея имеет жизнь, но только если добавить "Небинарный" (без указания пола)

@n1rwana n1rwana restored the other-gender-options branch October 13, 2022 08:42
@n1rwana n1rwana reopened this Oct 13, 2022
@n1rwana
Copy link
Member Author

n1rwana commented Oct 13, 2022

Resolves #748

@NoPlagiarism
Copy link
Contributor

Если всё-таки собрались менять sex на gender. То как минимум, надо поменять ещё дефолтные fields в VKAPI

@celestora celestora force-pushed the master branch 3 times, most recently from a9d021b to cbec4b5 Compare February 7, 2023 20:46
@openvk-org openvk-org force-pushed the master branch 5 times, most recently from e198aff to 2db815d Compare June 15, 2023 12:21
@mrilyew
Copy link
Contributor

mrilyew commented Oct 31, 2024

бра

@@ -72,6 +72,10 @@
"gender" = "Gender";
"male" = "male";
"female" = "female";
"non-binary" = "non-binary";
"other" = "other";
"other_gender_placeholder" = "I refer to myself as...";
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

identify as*

@@ -52,7 +52,7 @@ function get(string $user_ids = "0", string $fields = "", int $offset = 0, int $
$response[$i]->verified = intval($usr->isVerified());
break;
case "sex":
$response[$i]->sex = $usr->isFemale() ? 1 : 2;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

мб оставить для совместимости? isFemale поменять на ?bool и возвращать NULL, если не кун и не тян

@@ -0,0 +1,14 @@
<?xml version="1.0" encoding="UTF-8"?>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ненужно

@mrilyew mrilyew changed the title Added other gender options feat(profile info): add custom genders Dec 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
7 participants