hdgarrood / redmine_release_notes

A plugin to add release notes to Redmine
GNU General Public License v3.0
73 stars 46 forks source link

For the popular issue #75 (release note generation under the Version's page) #80

Closed dipanm closed 11 years ago

dipanm commented 11 years ago

This is a first check-in take a look at the patch, it's a full new functionality and also provide me code feedback.

We need to discuss about finalizing the stuff.

Here are todo/open discussion:

  1. debate/finalize the look and feel and details on dashboard.
  2. Release note dashboard should come up only if the Release note module is enabled for the project.
  3. Should the Release note tab disappear now?
  4. I would like that the entire dashboard div should be able to hid/show with click on the title of it. (is this a good idea?)
  5. Should the 'Generation can also be below the same div'?
  6. Need to add locales other than en.
  7. Any re-factor as needed.

Any feedback would be welcome.

hdgarrood commented 11 years ago

Also, it would be nice if your commits had your name and email on them. Or at least GitHub username, so that we can look back in the logs and see who made which changes. https://help.github.com/articles/set-up-git tells you how

dipanm commented 11 years ago

OK, things looks good now. I have added a TODO list in the check in list. Few are still the debates so please think about that. I will fix the balance stuff and make a proper stuff. I have done good testing as far as stats are concerned.

Please provide feedback.

hdgarrood commented 11 years ago

The balance stuff?

This looks great so far, so thank you.

Re 6 on the TODO list, since the issues aren't displayed by default after the page loads, I think that's the right approach. It might also be worth having a link to an the issues list with the appropriate filters set, so that it can be bookmarked/saved as a custom query? Re 7, I'm not sure. I'd have to play with both types in order to make a decision.

hdgarrood commented 11 years ago

Re 8, that could be done with AJAX without significant internal changes, and I think it could work quite well.

dipanm commented 11 years ago

Here things looks good now. Here is the status:

  1. project wise enable / disable -- NOT in shape. All functionality- adding release notes in issue, dashboard and even generation of release notes - works even if module is off for a project.
  2. if all done - no table division indicating done list and '0' todo - DONE
  3. fix locales - DONE
  4. Update the warning messages based on new taxonomy of stats. - DONE.
  5. Release note tab to go - DONE (though the underlying code does exist)
  6. The issue wise details - for 'required' and 'not-required' part of the table can be made hide/display - DONE.
  7. the dash-baord itself can be hide/expand - DONE.
  8. "generate release notes" can as well be opened up below the version page rather than new - I don't know how to do it.

Apart from this, I have done some clean ups. Please finalize this and take this if seems perfect; Please give feedback for anything that needs to be fixed..

hdgarrood commented 11 years ago

Awesome :)

Could you open a separate issue for your point 8? Could you also explain it a bit more -- I'm not quite sure what you mean.

Re 1, are you happy to do that, or shall I?

I'm happy with how the code looks; all that's left (aside from the above) is for me to try out your changes and see how they look from the user's point of view.

dipanm commented 11 years ago

I have updated the tracker query formation as you suggested. You are right that jQuery would be useful, but I didn't knew how to add dependencies in the redmine plugin. If you can help we can do this.

I am out of idea on point 1 right now - so I would like you to pull this out.

Regarding point 8 i meant that, should clicking "Generate .." would expand a DIV below or will it be a new page? I am good either way. I thought that may be more difficult to make it expandable like the dashboard because it needs to be from a completely different controller. Anyway, i think it is best to deal with that as a separate issue.

I now submit this - so please take over and bring it to fruition.

hdgarrood commented 11 years ago

Cool, ok. Thanks again :)

Incidentally Redmine includes jQuery, so it's safe to assume it's already there.

hdgarrood commented 11 years ago

Merged into feature branch rearrange-gui for further work until this is ready for master.