[07:33] good morning [12:40] mhall119, davidcalle: do you think at some stage you're going to have at least time for a cursory review of https://code.launchpad.net/~dholbach/developer-ubuntu-com/rework-importer/+merge/281745? [12:40] (you can ignore large parts of the tests data, they're just imported snappy/snapcraft docs) [12:51] dholbach: nice image handling! [12:53] dholbach: I've reviewed most of it already, but never got around to test it fully, I'll do my best to do it between today/tomorrow [12:53] davidcalle: sure, no worries [12:53] thanks a lot for the reviews then :) [12:53] we could take a look at md_importer/TODO at some stage and see what still is important [12:54] I'l take a look at only-update-if-HTML-changed now [13:20] davidcalle: have you interacted with revisions of pages or text plugins in the devportal code already? [13:21] dholbach: hmm, look at the translation dashboard, there is a bit of it, but not much iirc [13:21] I can see plugin_change.changed_date [13:21] which could be something I could work with [13:21] I was just wondering if revisions were stored somewhere and we could do something with them [13:26] dholbach: they are, I know you can access them with raw db calls but that's not ideal of course, I had to look at the revisions code itself to manage to get things working (not sure if it was for an early version of the importer or for an early version of the translations dashboard). I don't have this code anymore, though. [13:26] ok, no worries [13:26] working with .changed_date will be good enough for now [13:46] davidcalle: it looks like some tags produced by the github markdown extension we use are automatically stripped by RawHTML/AbstractText [13:47] so I can't easily check if an update would change the text.. [13:49] I'm trying to avoid updating the pages, otherwise the translations dashboard will be useless for example [13:49] and although we can't easily check revisions right now, at some point in the future it'd be nice [13:52] ah, I might have found something... [13:55] dholbach: oh, right, that's annoying [14:05] ok, I worked around it, found another issue with the code though :) [14:41] davidcalle: https://developer.staging.ubuntu.com/translations-dashboard/ look good (other than CSS, I'm working on collectstatic now) [15:53] davidcalle: I guess you're not overly bored... but maybe you have an idea what's going on here: http://paste.ubuntu.com/14478553/ :) [16:03] dholbach: is it a diff to apply on the rework-importer branch? [16:03] yep [16:18] dholbach: hmm, maybe you could filter for only published pages/plugins? [16:18] ok... I can take a look at it tomorrow [16:21] dholbach: I don't know, my only guess is that it's listing a plugin other than what you have explicitely created, hence my suggestion [16:23] ubuntu-gsoc@lists.u.c [16:28] oops, sorry [17:02] davidcalle: so therehave been two problems with collect static [17:03] #1: the swift_proxy environment variable was being set, when it should have been left empty [17:03] that's fixed [17:03] #2: the mojo predeploy script was trying to replace swift variables in developer.staging.ubuntu.com_http.conf but that's not just a lightweight redirect to developer.staging.ubuntu.com_https.conf so I'm having to change the predeploy [17:04] that's fixed, but I'm trying to get mojo to push out the new files to the current devportal-apache instances [17:25] davidcalle: can you check that the css fixed to the subnav are working on https://developer.staging.ubuntu.com/ now? [20:28] mhall119: yep, working great! [20:29] davidcalle: great, I've filed an RT to update production, but the JS scopes docs won't be imported until they're in the sdk team's release ppa [20:34] mhall119: looks like something went wrong with the move from staging>release PPA, I'll ping bzoltan first thing tomorrow [20:35] ok, imports run overnight, so if he fixes it tomorrow it won't show up until thursday === JoseeAntonioR is now known as jose