milgner / redmine_airbrake_server

A Redmine plugin which enables it to receive application error notifications Airbrake-style
28 stars 10 forks source link

configure default fixed version setting #8

Closed alexpooley closed 12 years ago

alexpooley commented 12 years ago

We develop systems in short iterations. We store these as versions in redmine. In parallel we also have a "hot fix" version which takes priority over all other work. This patch allows you to configure a default redmine version for the created issue. In our case we've hooked this up to the "hot fix" version.

milgner commented 12 years ago

Thanks for the contribution! This is indeed a good idea. And it's always nice to know that ones OSS is used by others, too :) I'm now thinking that maybe some settings should be moved into the Redmine webinterface instead of configuring them per-client. Also, I added the feature to the integration test in commit 11d54905262f18d43b48a4024eb506625b670e98

alexpooley commented 12 years ago

I think integration with Redmine UI would help a bit - everything but the API Key? I got caught on case sensitive project name, and copy+paste error from the README, so point and click would be nice :) Also, it would be good to use strings rather than IDs in the config file as IDs are not very intuitive.

Either way, great plugin! Helps us a lot.

milgner commented 12 years ago

Thanks, that's great to hear! I opened a corresponding ticket #9 and will put this together shortly.

milgner commented 12 years ago

Just a quick update: I implemented configuration via webinterface, but it was a bit ugly. When I discovered there was no 'nice' way to extend the project settings page, I decided to fall back on an existing plugin for that. It is a very similar hack to what I came up with, but at least it's encapsuled in its own plugin and thus cleaner code in this one. Also I think there's less chance of breaking things with regards to other plugins that may want to extend the project settings page ^^ I ran the tests and did a little bit of manual testing, too, so I'm fairly confident everything works... If there's any problems or enhancement requests, feel free to submit a ticket and/or pull request :) Cheers!

alexpooley commented 12 years ago

Thanks for the update, I'll check it out soon. We've had some troubles with hop toad notifier in production since upgrading to rails 3.1 (REE 1.8.7)

We were getting sporadic catastrophic failures in production until I removed the hop toad notifier gem. Have you seen this before? Any idea what might be causing this?

https://gist.github.com/57f75a64d4cd6c07dc2d

milgner commented 12 years ago

I heard that Rails 3.1 prefers Ruby 1.9 but as far as I know it should still work fine with REE for now. Are there any logs on the server side? Also, you could try updating to the current version of the notifier https://github.com/airbrake/airbrake