Roel Janssen <r...@gnu.org> skribis: > I've attached a patch that adds status icons to the links in the > expanded view. It only loads when the user clicks on "expand".
This is great! Minor things: I would move the patch list above the status, and remove the “status:” heading. > We might need to add a HTTP header from the GNU server to allow > loading data from external sources: > > https://en.wikipedia.org/wiki/Cross-origin_resource_sharing Indeed, because here the requests get blocked. Could you get in touch with webmast...@gnu.org and report back? > I hope my commit message is OK this way. It’s almost perfect. ;-) The only missing bits are the changes to packages.css that are not mentioned, and the added files that must be listed. > For fetching all statuses in one API call, I would like to propose > adding a function to Hydra that can return the status in an array like > the following JSON code: > > [ > > {"timestamp":1447890755,"buildstatus":0,"job":"a2ps-4.14.x86_64-linux","jobset":"master","project":"gnu","system":"x86_64-linux","nixname":"a2ps-4.14","finished":1,"id":812555}, > > {"finished":1,"id":811961,"system":"i686-linux","nixname":"a2ps-4.14","project":"gnu","jobset":"master","job":"a2ps-4.14.i686-linux","timestamp":1447890647,"buildstatus":0}, > > {"timestamp":1447890485,"buildstatus":0,"job":"a2ps-4.14.armhf-linux","nixname":"a2ps-4.14","system":"armhf-linux","id":811309,"finished":1,"jobset":"master","project":"gnu"}, > ... > ] > > Then I could add columns to the table with build status icons. I don't > know the Hydra code (or Perl), so I wonder if anyone else has the time > and wants to add the API call. I’d rather avoid hacking it, to be honest ;-), but I don’t mind if someone else does, of course. Thank you! Ludo’.
