Closed
Bug 932903
Opened 12 years ago
Closed 12 years ago
Add b2g desktop builds to pulse translator
Categories
(Webtools :: Pulse, defect)
Webtools
Pulse
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jgriffin, Assigned: jgriffin)
Details
Attachments
(1 file, 1 obsolete file)
|
15.84 KB,
patch
|
whimboo
:
review+
|
Details | Diff | Splinter Review |
We need to add a mapping for b2g desktop builds to the pulse translator, so that it will generate 'normalized' pulse messages for those builds.
| Assignee | ||
Comment 1•12 years ago
|
||
Comment 2•12 years ago
|
||
Comment on attachment 824771 [details] [diff] [review]
desktop_builds.diff
Review of attachment 824771 [details] [diff] [review]:
-----------------------------------------------------------------
Is that enough? I wonder if you could also add an example pulse message under test so I could check how it works.
| Assignee | ||
Comment 3•12 years ago
|
||
Added a test message.
Attachment #824771 -
Attachment is obsolete: true
Attachment #824771 -
Flags: review?(hskupin)
Attachment #826155 -
Flags: review?(hskupin)
Comment 4•12 years ago
|
||
Comment on attachment 826155 [details] [diff] [review]
desktop_builds.diff
Review of attachment 826155 [details] [diff] [review]:
-----------------------------------------------------------------
Looks good to me. Thanks Jonathan!
Attachment #826155 -
Flags: review?(hskupin) → review+
| Assignee | ||
Comment 5•12 years ago
|
||
| Assignee | ||
Comment 6•12 years ago
|
||
deployed
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•