]> sipb.mit.edu Git - ikiwiki.git/commitdiff
review
authorJoey Hess <joey@gnu.kitenet.net>
Fri, 27 Mar 2009 19:51:13 +0000 (15:51 -0400)
committerJoey Hess <joey@gnu.kitenet.net>
Fri, 27 Mar 2009 19:51:13 +0000 (15:51 -0400)
doc/plugins/contrib/po.mdwn
doc/todo/need_global_renamepage_hook.mdwn

index c39008466a759a5b43cd2546329c8167d0fc899e..a5e3375ce2c2d6ac7164afb74f3e352c245d9ec8 100644 (file)
@@ -328,3 +328,14 @@ but the po thing is supposed to work without these ugly `our`.
 Seems like it was the last unaddressed item from Joey's review, so I'm
 daring a timid "please pull"... or rather, please review again :)
 --[[intrigeri]]
 Seems like it was the last unaddressed item from Joey's review, so I'm
 daring a timid "please pull"... or rather, please review again :)
 --[[intrigeri]]
+
+> Ok, I've reviewed and merged into my own po branch. It's looking very
+> mergeable. I would still like to go over the `po.pm` code in detail and 
+> review it, but it's very complex, and I'm happy with all the changes
+> outside `po.pm`. 
+> 
+> * Is it worth trying to fix compatability with `indexpages`?
+> * Would it make sense to go ahead and modify `page.tmpl` to use
+>   OTHERLANGUAGES and PERCENTTRANSLATED, instead of documenting how to modify it?
+> 
+> --[[Joey]] 
index 69655f2e9dee9910277b25899d294d79550b7552..b065c82f3fdc9cf2c93f9b143373e3f23129c9a6 100644 (file)
@@ -107,3 +107,7 @@ hashes is desired, it could return the full set of hashes.
 >>> I've implemented this nice solution in my po branch, please review.
 >>> I'm slowly coming back to do the last bits needed to get my po and
 >>> meta branch merged.  --[[intrigeri]]
 >>> I've implemented this nice solution in my po branch, please review.
 >>> I'm slowly coming back to do the last bits needed to get my po and
 >>> meta branch merged.  --[[intrigeri]]
+
+>>>> It looks good. I made some small changes to it in my own po branch.
+>>>> Nothing significant really. If this were not tied up in the po branch,
+>>>> I've have merged it to master already. --[[Joey]]