Bug 10973

Summary: changing state of patch fails?
Product: [Yocto Project Subprojects] Patchwork/Patchtest Reporter: Patrick Ohly <patrick.ohly>
Component: PatchworkAssignee: Jose Lamego <jose.a.lamego>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: Medium CC: bluelightning, leonardo.sandoval.gonzalez
Version: unspecified   
Target Milestone: 2.3 M3   
Hardware: x86   
OS: Multiple   
Whiteboard:
OS type for building Yocto: --- Type of Regression: ---
Verified: Documentation change: No (bug/feature does not impact docs)

Description Patrick Ohly 2017-01-26 07:26:38 UTC
I'm logged in as "pohly" and looking at the patch https://patchwork.openembedded.org/series/4071/#

That patch has been merged, but the status is still "New". When changing the status to "Accepted" and clicking "Update", I get a blank page with https://patchwork.openembedded.org/series/4071/# as URL.

Reloading shows that the status hasn't changed.

Should I be able to change the status? If not, a proper error should be displayed.
Comment 1 Jose Lamego 2017-01-26 16:44:16 UTC
I'm looking into this. Issue is present at Series view only. Patches status and bundles can be edited at patch view.
Comment 2 Jose Lamego 2017-01-30 23:33:49 UTC
Two patches were developed and can be seen at:
https://lists.yoctoproject.org/pipermail/yocto/2017-January/034255.html
https://lists.yoctoproject.org/pipermail/yocto/2017-January/034256.html

these will be tested in patchwork-staging before implementing in production.
Comment 3 Jose Lamego 2017-01-31 22:15:50 UTC
I prepared a version 2 of the patches (fixing some javascript syntax errors), but the POST method used to update the patch status seems to be blocked by the nginx server (used both at patchwork-staging and production. I will consult Michael H. for options.
Comment 4 Jose Lamego 2017-02-07 20:24:38 UTC
Version 2 of patches were successfully implemented in patchwork-staging and can be seen at:
https://lists.yoctoproject.org/pipermail/yocto/2017-February/034396.html
https://lists.yoctoproject.org/pipermail/yocto/2017-February/034396.html