Bug 9904 - patchwork: show cover letter message in series created with pull request
Summary: patchwork: show cover letter message in series created with pull request
Status: RESOLVED FIXED
Alias: None
Product: Patchwork/Patchtest
Classification: Yocto Project Subprojects
Component: Patchwork (show other bugs)
Version: unspecified
Hardware: All Multiple
: High enhancement
Target Milestone: 2.2 M3
Assignee: Jose Lamego
QA Contact:
URL:
Whiteboard:
Depends on:
Blocks: 9941
  Show dependency tree
 
Reported: 2016-07-11 14:11 UTC by Jose Lamego
Modified: 2017-12-06 17:02 UTC (History)
4 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: No (bug/feature does not impact docs)

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Jose Lamego 2016-07-11 14:11:35 UTC 
The usual cover letter message is not displayed in series Details [example: 1] when a new series is created from patches submitted through pull request [example 2]. The message should be included the same as when git send email is used [example: 3].

[1] https://patchwork.openembedded.org/patch/126781/
[2] http://lists.openembedded.org/pipermail/openembedded-core/2016-July/123696.html
[3] https://patchwork.openembedded.org/series/181/
Comment 1 Jose Lamego 2016-07-11 14:13:29 UTC 
Corresponding github issue: 
https://github.com/dlespiau/patchwork/issues/190
Comment 2 Leonardo Sandoval Gonzalez 2016-08-12 20:44:33 UTC 
Jose, can you share the link to review the fix?
Comment 3 Jose Lamego 2016-08-12 20:50:15 UTC 
Pull request is available in Patchwork's github repo (Comment 1).

Commit can be seen at:
https://github.com/dlespiau/patchwork/pull/196/commits/dc2f32e16b0611b5e9ad928cc93dcbe2f088a9eb
Comment 4 Jose Lamego 2016-08-17 16:20:02 UTC 
A staging instance with production data/hardware was requested in order to test critical fixes or to temporarily implement them while they get merged to upstream:
https://bugzilla.yoctoproject.org/show_bug.cgi?id=10153
Comment 5 Jose Lamego 2016-08-31 14:51:52 UTC 
Fix has been successfully tested, waiting for upstream merge. I'm closing this bug now and will open a new bug to implement the fix in production either by updating Patchwork code from repository or pointing to the new staging instance.