Documentation for multiple pronouns #118
Conversation
|
This is great @morganastra can this be brought in? I actually sat down to try and add a feature like this and didn't realize it existed until a last minute double check of the issue tracker. I'd actually prefer if the site didn't require the additional keyword and just accepted pronoun.is/they/he with the same behavior, but pronoun.is/they/:or/he is a much appreciated syntax! |
|
http://pronoun.is/they/.../themself/:or/he/:or/her doesn't seem to work correctly, I only see 2 out of 3 pronouncs 🤔 |
|
Added that as an issue, see #129 |
There was a problem hiding this comment.
Both changes look good and work in production. Can be viewed at https://pronounis.herokuapp.com for the time being, will be taken down when PR is merged.
|
It would be great to get this merged. I came to the issue tracker to request support for multiple pronouns, was happy to see that it was already supported, and that an MR was already up to document it. I was then disappointed to see that it went almost a year unmerged, but am happy to see recent activity. |
Resolves #117 . No feature added so no new testing necessary, everything already covered by existing tests (which have no reason to fail.)