ChiliProject is not maintained anymore. Please be advised that there will be no more updates.
We do not recommend that you setup new ChiliProject instances and we urge all existing users to migrate their data to a maintained system, e.g. Redmine. We will provide a migration script later. In the meantime, you can use the instructions by Christian Daehn.
Check for missing Journal code from the AAJ merge (Bug #406)
Description
- Check JournalsController missing code that wasn't merged
- Check Journal model for missing code that wasn't merged
- Check Journals views for missing code that wasn't merged
Related issues
related to Feature #123: Review and Merge acts_as_journalized | Closed | 2011-02-02 |
Associated revisions
[#406] Added missing and removed unused code for Journals
acts_as_journalized changed the interfaces and didn't include all of
the recent changes to the Journals system.
- removed Journal diffs, acts_as_journalized handles them differently
- fixed the Journal atom feed
- added the journal preview back in
- added missing tests
History
Updated by Felix Schäfer at 2011-05-13 11:14 pm
The JournalsController
is missing the index action, fix at https://github.com/finnlabs/chiliproject/commit/86ca7eb24d0091f3698f1d2562851579ce24a731.
Updated by Eric Davis at 2011-05-20 09:38 pm
git diff dabe5caa1ab4e4c5cc3ece50ff968d8b0c4be9a4^..dabe5caa1ab4e4c5cc3ece50ff968d8b0c4be9a app/controllers/journals_controller.rb
is showing a lot of other missing code. I suspect some view changes are missing too.
Updated by Eric Davis at 2011-05-20 09:48 pm
Model and helper look ok.
Updated by Eric Davis at 2011-05-20 10:46 pm
Did a review and merge of the missing code. Main changes:
- removed Journal diffs, acts_as_journalized handles them differently
- fixed the Journal atom feed
- added the journal preview back in
- missing tests
- Status changed from Open to Closed