Improve error message when using a URL with an incorrect repo name

RESOLVED FIXED

Status

Tree Management
Treeherder
P3
normal
RESOLVED FIXED
4 years ago
3 years ago

People

(Reporter: emorley, Assigned: KWierso)

Tracking

Details

Attachments

(1 attachment)

(Reporter)

Description

4 years ago
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

4 years ago
No longer blocks: 1030636
(Reporter)

Updated

4 years ago
Blocks: 1059400
Priority: P2 → P3
(Assignee)

Comment 2

4 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

4 years ago
Created attachment 8490180 [details] [review]
Fix the error condition

This seems to work locally.
Attachment #8490180 - Flags: review?(emorley)
(Reporter)

Comment 4

4 years ago
Comment on attachment 8490180 [details] [review]
Fix the error condition

lgtm - thank you :-)
Attachment #8490180 - Flags: review?(emorley) → review+
(Reporter)

Updated

4 years ago
Assignee: nobody → kwierso
Status: NEW → ASSIGNED
(Reporter)

Updated

4 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.