search

tentwentyfour / remaim

Redmine to Maniphest migration utility
MIT License
8 stars 2 forks source link

Add a progress indicator #5

Open kwisatz opened 8 years ago

kwisatz commented 8 years ago

Add a progress indicator to see what and if something is happening.

andymstone commented 7 years ago

This would be very helpful in diagnosing where problems are happening. As a quick and dirty solution I simply added the following at the beginning of the createManiphestTask function (currently line 507 of Wizard.php), so I could at least see which issue was causing failures:

print_r($issue);
kwisatz commented 7 years ago

@andymstone could you specify which problems were occurring. Actually, any unhandled problems should cause an exception to be thrown or at least an error message to be displayed.

andymstone commented 7 years ago

Thanks, the problem I was having was issue #19 which you've already dealt with.

I just left a comment here in case anyone else came looking for a quick way of adding some basic progress tracking. The exceptions are useful for knowing what is going wrong, but I couldn't see the Redmine issue number anywhere in the stack trace so I had no idea which issue was causing the exception.

kwisatz commented 7 years ago

OK, thanks. I'll see if I can't add that kind of information to some of the exceptions. For the progress iterator, it'll have to wait a little, I think it would make more sense for me to replace the entire CLI part (using an existing implementation, e.g. symfony/cconsole) then attempting to re-invent the wheel. But console integration doesn't appear to be trivial.