| davidcalle | dholbach: https://developer.staging.ubuntu.com/en/ click on Iot->Get started, then do it again | 08:50 |
|---|---|---|
| dholbach | https://developer.staging.ubuntu.com/en/snappy/snappy/get-started/as-dev/ | 08:51 |
| dholbach | looks like there's a snappy too much in the URL | 08:51 |
| davidcalle | dholbach: for some reason the menu path is relative (both in the dropdown and breadcrumbs) | 08:51 |
| dholbach | I was thinking if this was one of the redirects | 08:52 |
| dholbach | hum... I don't quite get it | 08:54 |
| dholbach | I removed the [x] advertise from two of the imports | 08:55 |
| dholbach | not sure if that's it though | 08:55 |
| dholbach | davidcalle, I'm working on https://code.launchpad.net/~dholbach/developer-ubuntu-com/importer-versioning/+merge/294202 now - it's working out quite nicely so far, only thing left to add is the alias logic (to add redirects) | 09:04 |
| davidcalle | dholbach: so, in the admin, we will be able to set an "alias" <-> "version" map? | 09:16 |
| dholbach | yes | 09:16 |
| dholbach | for any branch it can be configured | 09:16 |
| davidcalle | Nice | 09:17 |
| dholbach | davidcalle, my thinking right now is that we configure something like: <branch>, version=16.04, alias=current | 09:37 |
| dholbach | so that we get snappy/guides/16.04 and redirects from snappy/guides/current/ | 09:38 |
| dholbach | on the other hand would this create a big link/redirect/page farm | 09:38 |
| dholbach | maybe it'd make more sense to do this in the yet-to-come url-rewrite app(?) | 09:39 |
| dholbach | also making snappy/guides/X redirect to snappy/guides/current/X or something | 09:39 |
| davidcalle | dholbach: since there is no urgency for versioning (afaik at least), maybe it could be done through the url rewriting app | 09:45 |
| dholbach | so maybe I drop the 'alias' logic for now | 09:47 |
| dholbach | and we just add (if at all necessary) something to http://bazaar.launchpad.net/~developer-ubuntu-com-dev/developer-ubuntu-com/stable/view/head:/templates/404.html#L18 | 09:48 |
| dholbach | davidcalle, ^ what do you think? | 09:54 |
| davidcalle | dholbach: well, on the bright side, it's easy and painless to do. But on the other hand, we shouldn't start relying to much on it. Or if we do, at least make it faster. (eg. by dropping the check if the page we want to redirect exists or not). | 09:59 |
| dholbach | is the plan still that we replace it with an app(?) | 10:00 |
| dholbach | a django app I mean | 10:00 |
| davidcalle | dholbach: well, it would be better, stuart is already complaining we rely on javascript for this :p and it can be a bit slow (between 0 and 2 sec), making people think there is no page at all. | 10:01 |
| dholbach | right | 10:01 |
| dholbach | but with the options of having a redirect app or a javascript redirect in place, I'm wondering if adding 'alias' to the branch configuration makes sense at all :) | 10:02 |
| davidcalle | right | 10:02 |
| dholbach | davidcalle, I think I'll drop it for now | 10:03 |
| dholbach | once we have the redirect app, we could add a redirect.add() call :) | 10:04 |
| davidcalle | dholbach: I was trying to figure out how it would work in practice, yes | 10:04 |
| dholbach | we're not in a real hurry with that one I guess :) | 10:06 |
| dholbach | my mom and my niece are here for lunch now - I'll remove the alias logic afterwards again :) | 10:06 |
| davidcalle | dholbach: enjoy :) I'll have a few things to ask you as well ! | 10:07 |
| dholbach | cool | 10:09 |
| dholbach | davidcalle, https://developer.staging.ubuntu.com/en/snappy/get-started/as-dev/ - nice work! :-) | 12:28 |
| davidcalle | dholbach: hah, thanks. I've found a workaround for the eight-col auto resize the importer applies. There is a pr waiting for didier. | 12:35 |
| dholbach | <3 <3 <3 | 12:36 |
| davidcalle | dholbach: my question from earlier was actually your feedback on ^ ;) | 12:39 |
| dholbach | it's brilliant work :) | 12:39 |
| davidcalle | dholbach: I'm looking for your redirect-app branch and can't find it | 12:39 |
| dholbach | it was WIP and I think I remember mhall119 saying that it wasn't the direction we agreed on | 12:41 |
| dholbach | I think I deleted it as your 404 branch came up at the same time | 12:41 |
| davidcalle | Ah right | 12:42 |
| davidcalle | dholbach: alright. So, until we make an app for this: what do you think about speeding up the current redirecter? I'd like your thoughts on the following: | 12:43 |
| davidcalle | Currently, it updates the url, looks if the new one exists, redirects. The 2nd step makes things slow. Maybe we can get rid of it? If the redirect target page doesn't exist, you would still end up on a 404, but with an updated url. (as opposed to currently: if the redirect target is missing, you stay on the original 404). | 12:45 |
| davidcalle | Now that I think again about it: the "report a bug" button would make you report a bug on the updated url, which is where we would want the fix. | 12:46 |
| davidcalle | So, it wouldn't be too bad | 12:46 |
| dholbach | I think that makes sense | 12:47 |
| davidcalle | alright, thanks | 12:48 |
| davidcalle | dholbach: if you can have a quick look https://code.launchpad.net/~davidc3/developer-ubuntu-com/faster-404-redirects/+merge/294240 | 13:10 |
| dholbach | LGTM | 13:11 |
| davidcalle | ty | 13:12 |
| dholbach | davidcalle, http://docs.django-cms.org/en/latest/how_to/extending_page_title.html looks very nice to me - I'll play around with it :) | 13:58 |
| davidcalle | dholbach: here is our solution for tagging pages! | 14:01 |
| dholbach | yep | 14:01 |
| dholbach | davidcalle, when you talk to didrocks to merge the PR from you, can you get him to do the other one as well? :-) | 16:35 |
| dholbach | https://code.launchpad.net/~dholbach/developer-ubuntu-com/enable_admin_style/+merge/294275 | 16:48 |
| dholbach | ^ most important MP of the day | 16:48 |
| dholbach | all right... and with that I call it a day - see you tomorrow! :-) | 16:48 |
| minot | hi, I'd like to update the page https://wiki.ubuntu.com/UserDirectoryPHP to say that from 16.04, the default to use is /etc/apache2/mods-available/php7.0.conf | 19:28 |
| minot | as opposed to php5.conf | 19:28 |
| minot | It says immutable page and so I can't edit it. Apparently I can edit the Home page but I don't want to try it | 19:30 |
| pleia2 | minot: unfortunately that wiki had to be locked down due to spam, can you send your comment to the ubuntu-doc@lists.ubuntu.com mailing list? | 19:49 |
| minot | OK will do | 19:49 |
| pleia2 | (though that's a very old page and documentation is supposed to be on help.ubuntu.com/communty wiki, not sure how it ended up there...) | 19:49 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!