Closed Bug 1347287 Opened 7 years ago Closed 7 years ago

Wire up the Test-Centric-UI to live data

Categories

(Tree Management Graveyard :: Treeherder: Test-based View, enhancement, P1)

enhancement

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: camd, Assigned: camd)

References

Details

Attachments

(1 file)

This should use the PushID URL param to determine which Push to get data for.  Then fetch the list of manifests and Push info to display to the user.
Attached file hook up to live data
Attachment #8847269 - Flags: review?(eperelman)
Assignee: nobody → cdawson
Status: NEW → ASSIGNED
Priority: -- → P1
Blocks: 1337488
Comment on attachment 8847269 [details] [review]
hook up to live data

I forgot to make one change.  Clearing the review till I have that done.
Attachment #8847269 - Flags: review?(eperelman)
Sounds good. I've got a review in progress, just ping when you're ready for me to finish.
Comment on attachment 8847269 [details] [review]
hook up to live data

Ok, switched to ``await`` instead of ``.then`` all over.  Ready for review.
Attachment #8847269 - Flags: review?(eperelman)
Eli: OK, this is ready for you to check again.  I added my changes in a new commit that I'll squash when we merge, of course.  

Thanks again for all your tutoring.  :)
Comment on attachment 8847269 [details] [review]
hook up to live data

Eli merged this, so marking as r+ for him.  :)
Attachment #8847269 - Flags: review?(eperelman) → review+
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Component: Treeherder → Treeherder: Test-based View
Product: Tree Management → Tree Management Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: