1450032 - Convert bottom (secondary) nav bar to ReactJS

Status: RESOLVED FIXED

(Tree Management :: Treeherder: Frontend, enhancement, P1)

Description

[Cameron Dawson [:camd]]

Part of the overall conversion to ReactJS.

Estimated difficulty of 6

Comment 1

[GitHub Bugzilla PR Linker]

Attachment: Link to GitHub pull-request: https://github.com/mozilla/treeherder/pull/3779

Comment 2

[Cameron Dawson [:camd]]

Comment on attachment 8991455 [details] [review]
Link to GitHub pull-request: https://github.com/mozilla/treeherder/pull/3779

A couple selenium tests are still failing.  It sure looks like they SHOULD be passing.  :)  I'll reassign when I've got them passing.  Sorry for the false start.

Comment 3

[Cameron Dawson [:camd]]

OK, tests are passing now.  Sorry (again) in advance for how large this is...

Comment 4

[Treeherder GitHub Bugbot]

Commit pushed to master at https://github.com/mozilla/treeherder

https://github.com/mozilla/treeherder/commit/bc4e8a7b1436f04681b5ca0d7f9d78fc302000bb
Bug 1450032 - Convert bottom (secondary) nav bar to ReactJS (#3779)

Also convert term resultState back to resultStatus

I had wanted to migrate to using "resultState" instead, as it
seemed more descriptive of what it is.  But the filter params
are using "resultStatus" and it would not be worth
the effort to migrate.  It doesn't really matter, but I want to be
consistent to remove confusion, so moving these terms back
to "resultStatus"-ish names.