Success

User tests: Successful: Unsuccessful:

avatar elkuku
elkuku
5 Aug 2013

Re-add the missing code for issue relations as discussed in #137

What I really don't like here are the methods added to our Twig renderer extension thingy which is growing badly :( ...
I remember @b2z posted a solution but I don't remember where... Maybe worth some refactoring here before v1 ?

avatar elkuku elkuku - open - 5 Aug 2013
avatar elkuku elkuku - open - 5 Aug 2013
avatar b2z
b2z - comment - 6 Aug 2013

Well for me it is not such a big issue. Twig extension is designed to serve as a place for often used code. What we can do is to separate them into several extensions.

I think I found the discussion you was talking about. It was about modules/widgets.

BTW it's worth to look into @fabpot 's Extensions library. There is an internationalization support also (#73).

avatar - close - 11 Aug 2013
avatar mbabker mbabker - reference | c859232 - 11 Aug 13
avatar mbabker mbabker - merge - 11 Aug 2013
avatar mbabker mbabker - close - 11 Aug 2013
avatar elkuku elkuku - head_ref_deleted - 11 Aug 2013

Add a Comment

Login with GitHub to post a comment