1444986 - Wrong Initials of developer/author of patch/commit/revision if first letter is non-ASCII/unicode character

Status: RESOLVED FIXED

(Tree Management :: Treeherder: Frontend, defect, P3)

Description

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://treeherder.mozilla.org/#/jobs?repo=autoland&revision=b7ad4ad9f1c015a8ea0cd9fc860b4eb1d334213d&filter-resultStatus=testfailed&filter-resultStatus=busted&filter-resultStatus=exception&filter-resultStatus=retry&filter-resultStatus=usercancel&filter-resultStatus=runnable has "EL" as the initials for the patch author "Emilio Cobos Álvarez" because non-ASCII characters get filtered out: https://github.com/mozilla/treeherder/blob/d5e3436e45cf9667d96ced9f96cdf64f86ad5427/ui/job-view/Revision.jsx#L8

Once bug 1361876 get implemented in Firefox, this will be easier to implement.

Comment 2

[Joel Maher ( :jmaher ) (UTC -8)]

wfm:
https://treeherder.mozilla.org/jobs?repo=autoland&searchStr=emilio&author=ealvarez%40mozilla.com

Comment 3

[Jonathan Kew [:jfkthame]]

(In reply to Joel Maher ( :jmaher ) (UTC -8) from comment #2)

wfm:
https://treeherder.mozilla.org/jobs?repo=autoland&searchStr=emilio&author=ealvarez%40mozilla.com

When I view this, I consistently see "EL" for Emilio's initials, which is the issue that was reported here.

Re-opening, as this does not seem to WFM as of now. Emilio's initials should be shown as "EÁ".

Comment 4

[Jonathan Kew [:jfkthame]]

See https://github.com/mozilla/treeherder/pull/7884 for a PR that should fix this, afaics.

Comment 5

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://github.com/mozilla/treeherder/commit/3ea88f62d770e53ccbb758994e424226ab0ca8cb