mbasset / project_overview

Adds a new project overview page to redmine 2+.
MIT License
13 stars 9 forks source link

View Cleanup #15

Open CodeInsider opened 11 years ago

CodeInsider commented 11 years ago

This should be a task for the next release. Many views have many logic parts inside their code. This should be extracted to the helper or controller, depending of what code type it is.

This would improve the readability of the views itself a lot.

mbasset commented 11 years ago

Agreed! Most of those logic pieces should have been in a helper or model extension. On 2013-09-09 1:32 PM, "Daniel Felix" notifications@github.com wrote:

This should be a task for the next release. Many views have many logic parts inside their code. This should be extracted to the helper or controller, depending of what code type it is.

This would improve the readability of the views itself a lot.

— Reply to this email directly or view it on GitHubhttps://github.com/mbasset/project_overview/issues/15 .