Closed
Bug 945850
Opened 12 years ago
Closed 12 years ago
Add comment metadata to private cluster
Categories
(Testing :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: ekyle, Unassigned)
References
Details
Attachments
(1 file)
|
20.58 KB,
patch
|
mcote
:
review+
|
Details | Diff | Splinter Review |
This might be useful, but the public cluster already has all comments
| Reporter | ||
Comment 1•12 years ago
|
||
The test reference files have been renamed to better reflect the symmetry of the ETL now that the comments are in both public and private.
The private cluster has no comment content, only metadata. Here is an example:
"4605": {
"bug_id": 1045,
"comment": "[screened]",
"comment_id": 4605,
"isprivate": 0,
"modified_by": "dbaron@dbaron.org",
"modified_ts": 914357279000
}
Changes:
https://github.com/klahnakoski/Bugzilla-ETL/commit/2c7cf6acab8df7dbf616515310e1a1f0ab19b7f6?w=1
Test:
https://github.com/klahnakoski/Bugzilla-ETL/commit/2c7cf6acab8df7dbf616515310e1a1f0ab19b7f6?w=1#diff-2464b031c4302f5ea65c026afea8e467R168
Attachment #8351190 -
Flags: review?(mcote)
Comment 2•12 years ago
|
||
Comment on attachment 8351190 [details] [diff] [review]
Code changes only, see github for moved files
Review of attachment 8351190 [details] [diff] [review]:
-----------------------------------------------------------------
Looks good to me.
Attachment #8351190 -
Flags: review?(mcote) → review+
| Reporter | ||
Updated•12 years ago
|
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•