Closed cfchris closed 11 years ago
Looks like backwards compatible and worthwhile additions to me. :) :+1:
Merged. I'll also be porting the changes to the 1.8 branch.
Thanks!
On Mon, May 20, 2013 at 5:07 PM, Adam Tuttle notifications@github.comwrote:
Looks like backwards compatible and worthwhile additions to me. :) [image: :+1:]
— Reply to this email directly or view it on GitHubhttps://github.com/oarevalo/BugLogHQ/pull/34#issuecomment-18181589 .
Oscar Arevalo http://about.me/oarevalo http://www.coldbricks.com - Content Management System http://www.oscararevalo.com - My Blog
I had some bugs that I was trying to log with the default CF message. They were failing to be processed in the queue. It was because they were around 300 characters. And for some reason the SQL Server scripts specify "Message" as half the size of the others. So, I updated both SQL Server scripts.
I also needed to be able to call "notifyService()" with an application name. So, I added a cfargument that is defaulted to what was being used. This should be totally backwards compatible.