1450039 - Convert Models for accessing API data 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 5

Comment 1

[Cameron Dawson [:camd]]

I've decided to do many of these sooner rather than later because it makes some of the other work simpler.

Comment 2

[GitHub Bugzilla PR Linker]

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

Comment 3

[Cameron Dawson [:camd]]

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

Thanks!  :)

Comment 4

[Treeherder GitHub Bugbot]

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

https://github.com/mozilla/treeherder/commit/eddbc1c8a773d0d9ab553a32bf9b943b5416785a
Bug 1450039 - Convert Models for accessing API data to Classes (#3568)

Comment 5

[Ed Morley [:emorley]]

https://github.com/mozilla/treeherder/pull/3568#discussion_r191384529

Comment 6

[GitHub Bugzilla PR Linker]

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

Comment 7

[Treeherder GitHub Bugbot]

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

https://github.com/mozilla/treeherder/commit/8014eb96184a309eb6eae13f1ecc12c64490b131
Bug 1450039 - Add missing null-check before cancelling fetch promise (#3590)

Comment 8

[Cameron Dawson [:camd]]

Fixed