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/salutation neutral options #260

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Michael-Schaer
Copy link
Contributor

Genderneutrale Option für die Anrede.

@Michael-Schaer Michael-Schaer force-pushed the feat/salutation_neutral_options branch from 38fb9ae to dcd78c5 Compare March 25, 2023 12:08
Copy link
Member

@carlobeltrame carlobeltrame left a comment

Choose a reason for hiding this comment

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

Inwiefern unterscheidet sich diese neue Anrede von der existierenden Anrede "Hallo greeting_name"?

@Michael-Schaer
Copy link
Contributor Author

@carlobeltrame greeting name macht first_name.presence || nickname.presence || last_name.presence || company_name

Das ist für die Anrede eigentlich gar nicht so passend. "Hallo Meier" oder "Hallo Spenglerei Muster AG" ist ja nicht ganz korrekt. Ich denke, als Standardeinstellung kann man das so belassen, weil dann sicher ein Wert kommt. Wenn ich aber aktiv "Hallo [Vorname]" auswähle, ist es dann auch ok, wenn da nur der Vorname abgefragt wird.

Alternativ könnte man beim Pfadinamen als Fallback den Vornamen ausgeben und vice versa. Das war bisher aber noch nicht abgedeckt.

Copy link
Member

@carlobeltrame carlobeltrame left a comment

Choose a reason for hiding this comment

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

Ich bin immer noch nicht ganz sicher ob es nicht besser wäre, die default Salutation zu verbessern (den greeting_name durch etwas sinnvolleres zu ersetzen was intelligent entweder Pfadiname oder Vorname nimmt), und im Anrede-Dropdown einen Anzeigewert für diese Default-Variante einzufügen. Jetzt neu hat es 3 (statt vorher 1) Möglichkeit für eine Gender-neutrale Anrede. Bist du sicher dass das hier die beste Lösung ist? Ist dir überhaupt bewusst, dass man im Anrede-Dropdown schon bisher die leere, gender-neutrale Variante wählen kann?

@Michael-Schaer
Copy link
Contributor Author

Lukas hat ja noch darüber gesprochen, dass er ein zusätzliches Geschlecht (freiwillig aktivierbar) im Core vorschlagen würde. In diesem Fall würde ich wohl auf seine Lösung umstellen wollen.

Sonst haben wir hier eine Backup-Lösung. Die leere Variante heisst ja (mit etwas Abweichung), dass nichts gewählt wurde. Das hat schon eine andere Bedeutung als das aktive "ich möchte mit Hallo angesprochen werden".

@carlobeltrame
Copy link
Member

Also das heisst, du möchtest das jetzt schon mal vorab als zukünftigen Fallback reinnehmen? Oder noch abwarten ob von Lukas etwas kommt? Wenn Leute erst mal diese neuen Anreden ausgewählt haben muss man sie dann ja mittels Datemigration wieder ändern, falls man je eine Anrede entfernen will.

@Michael-Schaer
Copy link
Contributor Author

Genau, erst mal abwarten bitte!

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