Closed
Bug 1046226
Opened 9 years ago
Closed 9 years ago
Improve error message when using a URL with an incorrect repo name
Categories
(Tree Management :: Treeherder, defect, P3)
Tree Management
Treeherder
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: emorley, Assigned: KWierso)
References
Details
Attachments
(1 file)
Visiting: https://treeherder.mozilla.org/ui/#/jobs?repo=foo Results in: "Error retrieving job data!" Rather than: "Unknown repo!" This makes things like bug 1046225 harder to figure out.
Reporter | ||
Updated•9 years ago
|
No longer blocks: treeherder-sheriff-transition
Reporter | ||
Updated•9 years ago
|
Blocks: treeherder-dev-transition
Priority: P2 → P3
Reporter | ||
Comment 1•9 years ago
|
||
It looks like unfortunately this part: https://github.com/mozilla/treeherder-ui/commit/08cc4f1a6603b5fbf353091d6045166144f732e5#diff-84193e569f1b29f0ab46ad40005626f8R59 of bug 1065541 doesn't quite work. eg try: http://treeherder-dev.allizom.org/ui/#/jobs?repo=foo
Depends on: 1065541
Assignee | ||
Comment 2•9 years ago
|
||
(In reply to Ed Morley [:edmorley] from comment #1) > It looks like unfortunately this part: > https://github.com/mozilla/treeherder-ui/commit/ > 08cc4f1a6603b5fbf353091d6045166144f732e5#diff- > 84193e569f1b29f0ab46ad40005626f8R59 > > of bug 1065541 doesn't quite work. > > eg try: > http://treeherder-dev.allizom.org/ui/#/jobs?repo=foo Huh. I think I must've made a copy/paste fail somewhere in there. The conditions for showing the unknown repo error are: "result_sets.length == 0 && !isLoadingRsBatch.appending && !isLoadingJobs && !locationHasSearchParam('revision') && !locationHasSearchParam('repo') && !currentRepo.url I think the "!locationHasSearchParam('repo')" should be "locationHasSearchParam('repo')" without the "!" at the beginning.
Assignee | ||
Comment 3•9 years ago
|
||
This seems to work locally.
Attachment #8490180 -
Flags: review?(emorley)
Reporter | ||
Comment 4•9 years ago
|
||
Comment on attachment 8490180 [details] [review] Fix the error condition lgtm - thank you :-)
Attachment #8490180 -
Flags: review?(emorley) → review+
Reporter | ||
Updated•9 years ago
|
Assignee: nobody → kwierso
Status: NEW → ASSIGNED
Comment 5•9 years ago
|
||
Commit pushed to master at https://github.com/mozilla/treeherder-ui https://github.com/mozilla/treeherder-ui/commit/5cd5d9ea697faf6022210f79482d2344ae119cfd Bug 1046226 - Fix unknown repository condition
Reporter | ||
Updated•9 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Comment 6•9 years ago
|
||
Commit pushed to master at https://github.com/mozilla/treeherder https://github.com/mozilla/treeherder/commit/4aff201600349f7b848e2de4358641165662de20 Bug 1046226 - Fix unknown repository condition
You need to log in
before you can comment on or make changes to this bug.
Description
•