Closed Bug 1449240 Opened 7 years ago Closed 7 years ago

Convert Angular values.js to import-able constants

Categories

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

enhancement

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: camd, Assigned: camd)

References

Details

Attachments

(1 file)

These can be easily converted now.
Assignee: nobody → cdawson
Status: NEW → ASSIGNED
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Reopening since the PR hasn't landed
Status: RESOLVED → REOPENED
Priority: -- → P1
Resolution: FIXED → ---
Comment on attachment 8962823 [details] [review] Link to GitHub pull-request: https://github.com/mozilla/treeherder/pull/3378 What the? I have no idea how/why I would have marked that fixed. Must have fat-fingered it somehow... Thanks for reopening!
Attachment #8962823 - Flags: review?(emorley)
Attachment #8962823 - Flags: review?(emorley) → review+
Commit pushed to master at https://github.com/mozilla/treeherder https://github.com/mozilla/treeherder/commit/0b5fec254a6a6a8bb09cfa9c77d5daf9edf1d905 Bug 1449240 - Convert Angular values.js to import-able constants (#3378) After looking at these, I think I want to go back to using "th" prefix and "ph" prefix. It is a nice way to see right off the bat that these are constants that are established by our code and not some external library. So converting ``platformMap`` back to ``thPlatformMap``. It also makes for less code/blame churn as we move to imported constants. * Move thOptionOrder to constants.js * Move thFailureResults to constants.js * Move thRepoGroupOrder to constants.js * Move thFavIcons to constants.js * Move phCompareDefaultOriginalRepo to constants.js * Move phCompareDefaultNewRepo to constants.js * Move thDefaultRepo to constants.js * Move thTitleSuffixLimit to constants.js * Move phTimeRanges to constants.js * Move thDateFormat to constants.js * Move phDefaultTimeRangeValue to constants.js * Move phTimeRangeValues to constants.js * Move phBlockers to constants.js * Move phDefaultFramework to constants.js * Move phAlertSummaryStatusMap to constants.js * Move phAlertSummaryIssueTrackersMap to constants.js * Move phAlertStatusMap to constants.js * Move thJobNavSelectors to constants.js * Move thPerformanceBranches to constants.js * Move phDashboardValues to dashboard.js * Move phCompareBaseLineDefaultTimeRange to constants.js * Move thPinboardCountError to constants.js * Remove now-unused values.js * Move thResultStatusFilters to constants.js * Move thEvents to constants.js * Move thAggregateIds to a aggregateIdHelper.js * Move thReftestStatus to jobHelper.js * Remove now-unused provider.js
Status: REOPENED → RESOLVED
Closed: 7 years ago7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: