Saving notification config for one project clears it for the rest
Reported by Jon Yurek | December 16th, 2008 @ 01:32 AM | in 0.1.9
I have the campfire and email notification gems installed. I have 4 projects set up in integrity. I put in the notification config for one project and save. I do "select * from integrity_notifiers;" in sqlite and I see the config for that project. When I enter config for another project and do the same query in sqlite I get the config for the second project and not the first one. Going to the first project's edit page shows empty notification config fields.
I'm using the 0.1.4 integrity gem, and the latest gems for foca-integrity-email and defunkt-integrity-campfire. I have sqlite 3.6.6, sqlite-ruby 1.2.4 (not sure you use this but figured it was worth reporting in case of interactions), and do_sqlite 0.9.9.
Comments and changes to this ticket
-
ronin-278 (at lighthouseapp) December 17th, 2008 @ 06:58 PM
- Milestone set to “0.1.9”
- Assigned user set to “ronin-278 (at lighthouseapp)”
- State changed from “new” to “open”
-
Simon Rozet February 16th, 2009 @ 02:15 AM
- State changed from “open” to “review”
Unfortunately, I couldn't reproduce the bug Jon described. But, here is a branch with a test case that use multiple notifier. http://github.com/foca/integrity...
-
Simon Rozet February 16th, 2009 @ 02:15 AM
- Assigned user changed from “ronin-278 (at lighthouseapp)” to “Simon Rozet”
-
Simon Rozet February 24th, 2009 @ 05:06 PM
- Assigned user changed from “Simon Rozet” to “ronin-278 (at lighthouseapp)”
-
Simon Rozet February 26th, 2009 @ 03:25 PM
I definitely can't reproduce this. I'll clean-up the test and merge this in 0.1.9 later today
-
Simon Rozet February 28th, 2009 @ 01:06 AM
- State changed from “review” to “resolved”
This is fixed in http://github.com/foca/integrity/commit/72e9fa42c3ec2e0cd495cd723b3883b4c2ed7424.
Huge thanks to atmos for figuring this out!
Please Sign in or create a free account to add a new ticket.
With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.
Create new ticket
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป
Automated continuous integration server that doesn't suck.
People watching this ticket
Jon Yurek
Simon Rozet
Tags
Referenced by
-
48
Add acceptance tests that use all the notifiers
I couldn't reproduce #43 so this probably needs a bit mor...
-
48
Add acceptance tests that use all the notifiers
This will be fixed in #43