
The server should not be sending any strings that get rendered in the client, because these strings are in English and we can't be sure that the user on the other end has their client set to English (in fact a majority of people probably knows English only as a second language, or does not know it very well at all). In fact issue #444 already covers this, but I'm making another ticket just for this message because changing it would allow us to drop the humanize dependency.
Here is a link to the message:
|
raise ClientError((f"You are banned from FAF {ban_time_text}.\n " |
It should be replaced with something like:
{
"command": "banned",
"expires_at": <ISO timestamp>
}
See #183 for inspiration, but note that we should be sending ISO timestamps (in UTC) and not timedeltas.
IssueHunt Summary
Backers (Total: $5.00)
Tips
The server should not be sending any strings that get rendered in the client, because these strings are in English and we can't be sure that the user on the other end has their client set to English (in fact a majority of people probably knows English only as a second language, or does not know it very well at all). In fact issue #444 already covers this, but I'm making another ticket just for this message because changing it would allow us to drop the
humanizedependency.Here is a link to the message:
server/server/lobbyconnection.py
Line 1094 in e4ca993
It should be replaced with something like:
See #183 for inspiration, but note that we should be sending ISO timestamps (in UTC) and not timedeltas.
IssueHunt Summary
Backers (Total: $5.00)
Become a backer now!
Or submit a pull request to get the deposits!
Tips