File viewer - Let fileviewer accept any length revision number

[klahnakoski]

It is not obvious that the revision number must be 12 characters:

This works

https://charts.mozilla.org/coverage/#/file?revision=8cc2427a322c&path=js/src/wasm/WasmValidate.cpp

while this does not

https://charts.mozilla.org/coverage/#/file?revision=8cc2427a322caa1e2c09ca3957335f88e573dc7a&path=js/src/wasm/WasmValidate.cpp

[marco-c]

They both don't work for me.

[klahnakoski]

Here is a new link:
https://charts.mozilla.org/coverage/#/file?revision=36c84344de23&path=js/src/wasm/WasmValidate.cpp

For if you are looking for a revision, for this file

{
    "from": "coverage",
    "where": {"and": [
        {"gt": {"repo.changeset.date": {"date": "today-2day"}}},
        {"eq": {"repo.branch.name": "mozilla-central"}},
        {"eq": {"source.file.name": "js/src/wasm/WasmValidate.cpp"}}
    ]},
    "groupby": ["repo.changeset.id12"]
}

https://activedata.allizom.org/tools/query.html#query_id=BaBq3XsR

[armenzg]

I'm getting and XML issue when clicking on that first URL.

[klahnakoski]

I did not notice that this was the old charts link. Here is the heroku link

https://firefox-code-coverage.herokuapp.com/#/file?revision=36c84344de23&path=js/src/wasm/WasmValidate.cpp

but is still do not see a problem.