Redmine

+1

+1

• the view doesn't handle issues which have been assigned to a target version (which itself has a set due date) but are missing a specific start- and/or due date. Such results in an error like this:

  Processing IssuesController#gantt (for 192.168.1.2 at 2008-12-25 01:30:30) [GET]
    Session ID: a8fdc48242b0d8a394ae19afefca3730
    Parameters: {"project_id"=>"test1", "action"=>"gantt", "controller"=>"issues"}
  Rendering template within layouts/base
  Rendering issues/gantt.rhtml
  
  ActionView::TemplateError (You have a nil object when you didn't expect it!
  The error occurred while evaluating nil.<=) on line #271 of issues/gantt.rhtml:
  268: top = headers_height + 10
  269: @gantt.events.each do |i| 
  270:   if i.is_a? Issue 
  271:        i_left, i_width, l_width, d_width = get_position(i, @gantt.date_from, @gantt.date_to, zoom)
  272:     %>
  273:   <div id="ev_<%=i.id%>" style="position:absolute;left:<%= i_left %>px;top:<%= top %>px;padding-top:3px;height:18px;width:<%= i_width+100 %>px;" class="handle">
  274:     <div id="task_todo_<%=i.id%>" style="float:left; width:<%= i_width %>px;" class="task task_todo"> </div>
  

  
  which (for the due date only) can be fixed by using i.due_before instead of i.due_date in the view for these issues (see changes in r1455).
• followup of above: the view should handle issues with an assigned target version (which itself has a set due date) by using the due date from the targetted version as the fixed due date of the issue (so without calendar link).
• the gantt-objects shouldn't be draggable (to the right) when the issue due date is fixed by an assigned target version's due date. Only the start date should be changable by dragging it to the left.
• issue's due dates should be marked when it's linked against a target version's due date.

Looking at the above sum I'm guessing that I am probably able to implement the changes myself in some time. Probably most needed enhancements could be made by adding some nested if i.due_date?-structures. I'll see what I can do...

Greetings...

Hiroyuki Yoshioka wrote:

I'm sorry for the delay in my reply.
I overlooked your comment.

Never mind. Appreciate your efforts on this issue already... ;)

How about this patch?

The changes are below:

• The view can handle issues which have been assigned to a target version (which itself has a set due date).
• Issue's version is displayed when it's linked against a target version's due date.
• Check issue's dependencies and update other bar when the date is changed.

This seems to be solving most (or all) issues I've ran into when testing this. As soon as I have my devel-deployment updated to current trunk (currently stuck at r2167) I'll start some more tests on this patch...

+1

Maybe you should get some ideas from the ClokingIT code, if the license permits it.

Code : http://wiki.clockingit.com/wiki:source
Screenshots : http://www.clockingit.com/screenshots

There's a video showing the interactive gantt chart.

Mischa The Evil wrote:

Hiroyuki Yoshioka wrote:

How about this patch?

The changes are below:

• The view can handle issues which have been assigned to a target version (which itself has a set due date).
• Issue's version is displayed when it's linked against a target version's due date.
• Check issue's dependencies and update other bar when the date is changed.

This seems to be solving most (or all) issues I've ran into when testing this. As soon as I have my devel-deployment updated to current trunk (currently stuck at r2167) I'll start some more tests on this patch...

I've tested the updated patch against trunk at r2641 and it seems to work (very) fine by now. All the mentioned issues are handled.
Only one last thing that comes up by now: the changes in the issue's start-/due-dates aren't reflected as issue-journals [ - sic - ] project-activity.

HTH... ;-)

great feature ! one small bug : after moving an issue, dates are correctly updated (in the second column), but in the calendar the date is the previous value (not the new one).

It would be also nice to be able to change the start and end of an issue directly with the mouse.

Hello, how to install this patch in a Bitnami Redmine Stack in Windows??? I download a GNU Patch Utility but fail.

The same for me.

All design is broken in the gantt view.

Yohann Monnier wrote:

The same for me.

All design is broken in the gantt view.

I love the idea and the possibilities but it lacks stability on r2671 it works.
On other versions its broken by updates and other plugins.

Is there a chance that this would make it to the core? Or a plugin if that is not feasible?

Yohann Monnier wrote:

The same for me.

All design is broken in the gantt view.

Hmm..
I tested gantt-editting-r2671-2.patch on redmine trunk revision r2671 and r2819, but it works for me.
What kind of issues do you create?
And could you send a html source when the gantt chart is broken?

any news for the latest r2904?
It would be great to have the patch for the latest revision.

thanks in advance!

Hiroyuki Yoshioka wrote:

Hi, I made a patch for the r2924.
See the file gantt-editting-r2924.patch.

Thanks a lot! It is I needed for.

Just wanted to let everyone here know, I have a huge rewrite of the Gantt chart I'm working on. It has a bunch of new features and I removed a lot of duplicated code (HTML, PDF, and PNG rendering where duplicated). It's still being tested and could use some optimization but I'm hoping to have it ready in the next month or two.

Eric Davis wrote:

Just wanted to let everyone here know, I have a huge rewrite of the Gantt chart I'm working on. It has a bunch of new features and I removed a lot of duplicated code (HTML, PDF, and PNG rendering where duplicated). It's still being tested and could use some optimization but I'm hoping to have it ready in the next month or two.

It sounds great.
I'm looking forward to the release.
:)

Eric Davis wrote:

Just wanted to let everyone here know, I have a huge rewrite of the Gantt chart I'm working on. It has a bunch of new features and I removed a lot of duplicated code (HTML, PDF, and PNG rendering where duplicated). It's still being tested and could use some optimization but I'm hoping to have it ready in the next month or two.

sure, sounds very good,
waiting for the release :)

as for patch, it stopped to redraw chart when changed due_date from calendar element, and it is redrawn after refreshing the page with F5.

Eric Davis wrote:

Just wanted to let everyone here know, I have a huge rewrite of the Gantt chart I'm working on. It has a bunch of new features and I removed a lot of duplicated code (HTML, PDF, and PNG rendering where duplicated). It's still being tested and could use some optimization but I'm hoping to have it ready in the next month or two.

One remark about the (nice) plugin initialised by Hiroyuki Yoshioka : when moving an issue with the mouse, the modification is done immediately, so the issue's history is updated. This is fine if you are sure of what you are doing (for example task planning is already fixed and you just apply it in redmine). My point of view is that the gantt char edition feature using the mouse is especially usefull when you are trying to find the best planning for your tasks. In that case, you'll probably have to move each issue several times before reaching a satisfying planning. This will cause a huge list of "date changement" in the history, which is not relevant. My suggestion is to have a "validate" button which will apply the Gantt Chart modifications when we are satisfied with the planning.

This is a user feedback, I don't know if it easy to do ;-)

Eric Davis wrote:

Just wanted to let everyone here know, I have a huge rewrite of the Gantt chart I'm working on. It has a bunch of new features and I removed a lot of duplicated code (HTML, PDF, and PNG rendering where duplicated). It's still being tested and could use some optimization but I'm hoping to have it ready in the next month or two.

Hi Eric,
I hope you are doing well and successfully handle your multiple redmine projects...
Did you manage to move forward on this one? Hiroyuki's patch is now a vital feature for my team but I struggle each time I want to upgrade Redmine trunk, so your support in this (especially in including it in the trunk?) would be great...
Thanks for feedback!

Guillaume Pellegrin wrote:

Eric Davis wrote:

Just wanted to let everyone here know, I have a huge rewrite of the Gantt chart I'm working on. It has a bunch of new features and I removed a lot of duplicated code (HTML, PDF, and PNG rendering where duplicated). It's still being tested and could use some optimization but I'm hoping to have it ready in the next month or two.

Hi Eric,
I hope you are doing well and successfully handle your multiple redmine projects...
Did you manage to move forward on this one? Hiroyuki's patch is now a vital feature for my team but I struggle each time I want to upgrade Redmine trunk, so your support in this (especially in including it in the trunk?) would be great...
Thanks for feedback!

sure Eric, it would be very appreciated.
--andrew

I just realized how useful this feature would be during the planning of a project, especially if, as Loic suggested, it was possible to commit all changes at once when we're happy with actual plan. Hiroyuki, would it be possible to include this feature?

As for Eric's rewrite vs Hiroyuki's patch: would both be compatible, mutually-exclusive or redundant?

As always, thanks for all your work, guys!

Eric

loic Le Gallou wrote:

One remark about the (nice) plugin initialised by Hiroyuki Yoshioka : when moving an issue with the mouse, the modification >is done immediately, so the issue's history is updated. This is fine if you are sure of what you are doing (for example >task planning is already fixed and you just apply it in redmine). My point of view is that the gantt char edition feature >using the mouse is especially usefull when you are trying to find the best planning for your tasks. In that case, you'll >probably have to move each issue several times before reaching a satisfying planning. This will cause a huge list of "date >changement" in the history, which is not relevant. My suggestion is to have a "validate" button which will apply the Gantt >Chart modifications when we are satisfied with the planning.

Thank you for your suggestion.
It is good idea.

Eric Voisard wrote:

I just realized how useful this feature would be during the planning of a project, especially if, as Loic suggested, it was possible to commit all changes at once when we're happy with actual plan. Hiroyuki, would it be possible to include this feature?

It wouble be possible, but I think I should wait for Eric's rewrite.

Eric Voisard wrote:

As for Eric's rewrite vs Hiroyuki's patch: would both be compatible, mutually-exclusive or redundant?

They would probably work together but I've rearranged so much of the Gantt chart that there will be some merging needed.

Eric Hulser Davis : Where can i find your rewrite ? is this plugin will be integrated to the core of redmine ?

Eric Hulser Davis : Any news on the likely date of the new gantt chart and what the features will be ?

what about development (trunk) branch?
starting from r3695 gantts moved into the separated controller.
can anyone help to adopt the latest patch for trunk?

I tried but had no success. I have a leak of skills.

Any help will be appreciated.

Regards,
--andrew

any lucks so far?

Hi, I'm also very interested in this :-) I've moved my team over to redmine, using the redmine-sprints plugin... but the lack of Gantt chart planning means I'm still dependent on GanttProject :( Looking forward to hearing new news!

Hiroyuki Yoshioka wrote:

Hi, Andrew.
Try this patch.
See the file gantt-editting-r3764.patch.

Hi, Hiroyuki.

Your patch is perfect! It has been easily applied and tested. Your help appreciated much.

Thanks,
--andrew

Mood: thankful & happy

Did a quick review of gantt_edit_r3764.patch:

• missing tests
• logic in the controller should be moved to the moved
• javascript assumes Redmine is running at the domain root and uses a hardcoded url to '/issues'. Should use the Rails url helpers
• routing doesn't follow the REST patterns

Hi, Hiroyuki. and Eric

I want this Patch in Version 1.0.2 or 1.1.0.

How do this Patch?

and using want forever this Patch in next version
(ex : 1.2.x, 1.3.x, 2.0.x etc...)

Hi, seunghyun.

I will try to create a patch for 1.1.0.
Wait for several days.

Hi, Eric.

Thank you for your review.

I'm looking your new code.
Your rewrite is very good :)

Oh,damn
I so want to see this asa a part of the core of Redmine.
One of the few things RM lacks before beeing a complete project management web application.

I agree. This should be completed by some features (arrow between tasks for dependencies) and inserted in the core of redmine. This is a must have !

Thank you for your work !

seunghyun yi wrote:

Let me ask you one more thing.
When I modify Work schedules on Gantt Chart it updates the whole Redmine schedule real time. Is there an way to throw in an 'apply' button so the schdule only updates when I click it?

I'll think about it.
But it is a little bit difficult.
The problem is the method of caching data before apply without updating database,
and the collision with other users operation.

I've just run this patch against v1.1.1 as it works perfectly. Great addition to include show the start and due date as columns in the Gantt chart as well.

My only other recommendation would be to have a column to show Assignee (there may be a plugin or patch already that I'm not aware of).

The other outstanding improvement would be to allow administrators to specify the working week + holidays to avoid tasks getting scheduled on weekends or other non-working days. I see this is being suggested with feature #2161.

Thanks again for this great patch!

I've just looked through the roadmap for v1.1.2 and v1.2 and don't see this being incorporated into the core Redmine system. For me I feel this is a must have for anyone wishing to replace their existing project scheduling system and move over to Redmine and would strongly recommend it becoming part of the core application.

I'm fairly new to Redmine and these forums so can anyone explain how this review and incorporation process works.

Thanks.

If any task don't have due data this patch don't show the next dates, I test with redmine 1.1.1

In brief I put 2 new taks in the first In the two issues I write only the start date, and looking the Gantt I only look the start date of the first issue

If I try to move a bar and the task has not been created by me, the error message shows nothing and the task does not return to its position.

Tanks for your work

I have applied the latest patch 'gantt_edit_2_v1-1-1.patch' and completed some initial testing.

It has fixed the problem with the pop up error box, which is good. However, when changing either a start or due date it automatically
refreshes the screen and displays the dates incorrectly on the gantt chart. If I then manually refresh then it moves them to the correct date, so obviously it's saving it to the database OK.

The other problem is that I have a number of inter related tasks, which precede and follow one another. When changing the due date through the gantt interface, it doesn't re-adjust the dates on the related issues correctly. However, when changing the start date it does re-adjust the related issues correctly. In the previous version both changing the start date or the due would work fine, so it seems this last bug is something introduced in the latest version.

I have just tried the new 'Better Gantt Chart' plugin in conjunction with this patch and it seems to have resolved the problem with re-calculating dates for related issues. However, there are some compatibility issues between the plugin and this patch (see http://www.redmine.org/boards/3/topics/22288).

I hope this helps. I'm happy to give further feedback if you need. I am currently running 'Redmine 1.1.2.stable.5022 (MySQL)'

I really like this plugin and I can see it adds a huge improvement to Redmine. Please let me know if there is anything else I can do to assist the development of this plugin. As I mentioned in my forum post (22288) it would be great if the authors of this patch and the 'Better Gantt Chart' plugin could join forces somehow.

Chris Darts wrote:

I have just tried the new 'Better Gantt Chart' plugin in conjunction with this patch and it seems to have resolved the problem with re-calculating dates for related issues. However, there are some compatibility issues between the plugin and this patch (see http://www.redmine.org/boards/3/topics/22288).

Chris, as I replied to the forum post I have released a new version of the plugin which hopefully solves most compatibility issues.

As I mentioned in my forum post (22288) it would be great if the authors of this patch and the 'Better Gantt Chart' plugin could join forces somehow.

I'm all for it. I will continue update the plugin for compatibility with new versions of the patch.

Thanks for your quick response on this Alexey. As reported in the plugin forum post (http://www.redmine.org/boards/3/topics/22288) the new version of the plugin has resolved a number of the bugs, which is great.

Hiroyuki,

When using patch #2024 with the 'Better Gantt Chart' plugin, the current outstanding issues with the patch are as follows:

1. when changing either a start or due date it automatically refreshes the screen and displays the dates incorrectly on the gantt chart. If I then manually refresh then it moves them to the correct date, so obviously it's saving it to the database OK.
2. when you try to change the start or due date through the Gantt chart interface using the date picker, it automatically updates the date value (and therefore the bars on the Gantt chart) when you move between months, even if you haven't actually selected a date.

It would be great if it is possible to fix these 2 outstanding issues. I really feel that this patch, combined with the plugin greatly improve the standard functionality of the Gantt chart in Redmine. As Ivan Cenov added (http://www.redmine.org/boards/3/topics/22288?r=22292#message-22292) it would be great to see this functionality incorporated into the core Redmine system at some point in the future.

Thanks,

Chris

Is is possible to make this in a plugin, i think is better to update redmine.

Thanks

Yoshioka-san,

When using gantt_edit_2_v1-1-1.patch, the PNG export function with clicking right-under "PNG" link in gantt chart gives us "Internal Error".

The function this patch itself works fine. greeeeat!

Just I want to know, this is our own problem or not?

Would you tell me its OK or not in your environment.

Regards,

I have done small changes to avoid getting illegal html format output.

like following...

I am not sure everybody knows this problem or not.

Anyway, in {redmine}\lib\redmine\helpers\gantt.rb

def calendar_for_issue(issue, options)
  # Skip issues that don't have a due_before (due_date or version's due_date)
  if issue.is_a?(Issue) && issue.due_before

    case options[:format]
    when :html
+     @calendars << "<div style='position: absolute;line-height:1.2em;height:16px;top:#{options[:top]}px;left:4px;overflow:hidden;'>" 
      start_date = issue.start_date
      if start_date
-       @calendars << "<div style='position: absolute;line-height:1.2em;height:16px;top:#{options[:top]}px;left:4px;overflow:hidden;'>" 
        @calendars << "<span id='i#{issue.id}_start_date_str'>" 
        @calendars << format_date(start_date)
        @calendars << "</span>" 
        @calendars << "<input type='hidden' size='12' id='i#{issue.id}_hidden_start_date' value='#{start_date}' />" 
-       @calendars << "<input type='hidden' size='12' id='i#{issue.id}_start_date' value='#{start_date}'>#{view.g_calendar_for('i' + issue.id.to_s + '_start_date')}" 
+       @calendars << "<input type='hidden' size='12' id='i#{issue.id}_start_date' value='#{start_date}' />#{view.g_calendar_for('i' + issue.id.to_s + '_start_date')}" 
        @calendars << observe_date_field("i#{issue.id}", 'start')
      end
      due_date = issue.due_date
      if due_date
        @calendars << "<span id='i#{issue.id}_due_date_str'>" 
        @calendars << format_date(due_date)
        @calendars << "</span>" 
        @calendars << "<input type='hidden' size='12' id='i#{issue.id}_hidden_due_date' value='#{due_date}' />" 
-       @calendars << "<input type='hidden' size='12' id='i#{issue.id}_due_date' value='#{due_date}'>#{view.g_calendar_for('i' + issue.id.to_s + '_due_date')}" 
+       @calendars << "<input type='hidden' size='12' id='i#{issue.id}_due_date' value='#{due_date}' />#{view.g_calendar_for('i' + issue.id.to_s + '_due_date')}" 
        @calendars << observe_date_field("i#{issue.id}", 'due')
-       @calendars << "</div>" 
      end
+     @calendars << "</div>" 
    when :image
      #nop
    when :pdf
      #nop
    end
  else
    ActiveRecord::Base.logger.debug "GanttHelper#line_for_issue was not given an issue with a due_before" 
    ''
  end
end

I have applied patched gantt_edit_2_v1-1-1.patch to my 1.1.2 redmine, however I do not see any changes on the http://localhost:3000/projects/test/issues/gantt page, nor any arrows to adjust on the bar.
Do I need to wait for a 1.1.2 patch? Is there anything else that needs to be done on a vanilla 1.1.2 redmine besides the patch file?

See #7807.

Thank you, Chris.

• A popup dialog is displayed when due date of a project is out of range.
• When changing either a start or due date, it automatically refreshes the screen and displays the dates incorrectly in some cases.
• It tries to automatically change the start or due date when changing months on the date picker,
  without selecting a specific date.

I couldn't have made the patch without your cooperation.

• When changing a start or due date of a sub-task, it doesn't correctly show the adjusted start or due date for the parent task.

• the PNG export function with clicking right-under "PNG" link in gantt chart gives us "Internal Error".

Shibata-san,
thank you for your cooperation.

Hi,

Are there any plans to update this to work with version 1.2? Currently we are using the patch with version 1.1.3 and it works very well and I would like to update Redmine to 1.2 however we don't want to lose this functionality.

+1

+1 - I'd love to get this updated too

+1 too

+1 This one feature alone would push Redmine far ahead of the pack. This would really make it an awesome software development tracking system. And open source to boot!

+1 too
This feature working for new version would be very nice!

+1

To fix the PNG export 500 error:

1. Open \lib\redmine\helpers\gantt.rb
2. Line 579 gc.line(x, headers_heigth, x, headers_heigth + g_height-1)
3. Update to gc.line(x, headers_height, x, headers_height + g_height-1)

Is there any plan to include relationships between issues? Haven't been able to get this working yet.

Thank you

+1 Looking for this feature!

Hello,
I saw sources is that they are functional or is even the development period? Demand is still at 0% ?
What is the procedure for testing and installation?

+1

Really, to have and editable GANTT chart is something that would make REDMINE one of the best tools for Project management. It would be very appreciated to have this within the release version soon. I don't know if there is a better way for pushing for this idea.

same for me - drag'n drop support for Gantt charts is THE killer feature Redmine is currently missing... for my day to day work, it's really frustrating having to edit manually n tasks when I need to postpone one or another task in a project...

unfortunately, as a PHP specialist, I cannot help working on this, but I wish I could!

+1 !

This is really such a great function! I tried to patch 'gantt_edit_v1-1-2.patch' on both Redmine v1.3.2 & v1.4.3, but it seems not work, could you please have a check? Thank you so much!

There is no test. You need to add tests.

• source:tags/2.0.2/test/unit/lib/redmine/helpers/gantt_test.rb
• source:tags/2.0.2/test/unit/helpers/application_helper_test.rb
• source:tags/2.0.2/test/unit/helpers/issues_helper_test.rb
• source:tags/2.0.2/test/unit/issue_test.rb
• source:tags/2.0.2/test/functional/gantts_controller_test.rb

+1!
Nice patch! Can´t it be included on the next revision of Redmine?

+1

It is possible add this function to the core ?

+1, This function should be in the core.

+1 from me to add this to the core.

+1, changing due dates for many tickets (e.g. in case of sickness or some new, prioritized customer requirements) is a huge pain in the current version.

It is definately a "must have" feature! A perfect complement to the excludation of weekends as described in #2161.

I've just updated to Redmine 2.1.4.stable on rails version 3.2.8 and I'm having the troubles with the gantt charts not recalculating to the correct end dates. I see that the dates seems to be correct within the actual issue but not when viewing via the gantt chart. Is there a fix or patch for this version to get the gantt chart to work correctly?

Im working on 1.1.1, with Edit V5 1.1.1 patch, and better gantt patch for this version. Im also using Advanced roadmap, but when I add a version, roadmap work, but not gantt (all tab). If I delete version off my project, gantt appear.

Any help?

+1. Yes this is a great work. Hoping to see this.

+1

Daniel Felix wrote:

Hopefully the patches could be ported to 2.3 and finally find their way to the core.

The most important thing that is missing in the patches are the tests. They aren't there and with a feature like this that wouldn't be good...

This is clear to me. :-)
But first, the functionality must be ensured. After that, we could try to add some tests to it.

any plan/news for this feature?

Babak Khorrami wrote:

any plan/news for this feature?

I think it is dead already.
Wish to be wrong, but the guys at EasyRedmine already implemented the feature: http://www.redmine.org/plugins/advanced-drag-drop-gantt
It seems to be awesome, but it is very expensive... it is too much for startup teams...
This is the only reason our team still using Redmine 1.4.7

However we can offer some donation for this project, if somebody can complete it. (And maybe we are not the only one...)

Hi

Why not try our redmine client http://www.mana-sys.fr/en/redmineclient. This is an application, not a web plugin, but we include a first version of gantt chart editing. This is not as powerfull as easy redmine for sure and need some improvement but it works. It is not free but not so expensive. Have a look.

Olivier

Babak Khorrami wrote:

any plan/news for this feature?

+1

+1

hello,

is there any news for this feature please ?

forgive me, but those patches are against which redmine version please ?
are they working with the current 2.4 or 2.5 version ?

+1! Good for intuitively rearrange work packages etc

Hi everybody, are there any news about this feature? It would be awesome to have it.
Thanks.

1000+
This is very important feature.
All modern project systems supporting it.
It's a reason to move from redmine to something else.

I would still love to have this feature. Can someone organise to have some developers come together and create a kickstarter or similiar for a shared funding ?
I would certainly contribute in a meaningful way.

+1000

Thank you for your effort in providing and updating this patch! This feature and its UI implementation looks very promising and I would be happy to see this feature in Redmine soon.

After a quick review, I found some potential issues (in 0001-Moving-gantt-bar.patch however, which may need to be addressed:

• The new route is inconsistent with the rest of the routes. It should be under /issues/gantt, i.e.
  

    put '/issues/gantt/:id/change_duration', :to => 'gantts#change_duration', :as => 'gantt_change_duration'
  

• You allow to set any parameter or custom fields to the issue, not just duration related parameters. While the given params appear to be checked by safe_attributes (which thus probably doesn't cause security issues), this might still be surprising. Maybe it's useful to filter the allowed parameters?
• The view contains A LOT of code which mixes complex Ruby logic and Javascript template generation. It might be helpful to extract the Ruby parts into a helper and separate the generation of the Javascript response from the Ruby logic. This would also allow to actually test this logic.
• In the view, you appear to parse and update generated HTML response code of the Redmine::Helpers::Gantt object as well as directly access private method and instance variables. This is rather brittle (and slow and potentially insecure). Instead, it's likely a better path to update the class and add appropriate render methods directly.

Apart from that, it would be very helpful if you could include some tests:

• For the controller action, we should at least verify the happy path as well as handling of permissions, conflict handling, and validation errors
• The view / helpers should be tested in order to describe the desired behavior (which I find very hard to follow right now).

+1000

Holger Just Mizuki ISHIKAWA Can you review my patch?