search

twine-net / laravel-raven

Sentry (Raven) error monitoring for Laravel 4 & 5 with send in background
17 stars 2 forks source link

Add locale processor #18

Closed tremby closed 8 years ago

tremby commented 8 years ago

This processor adds a tag 'locale' with the app's current locale to an outgoing report.

pulkitjalan commented 8 years ago

This can no longer be merged :(

tremby commented 8 years ago

Yes, that was expected, since they both added the same section to the readme, but with different subsections. I've rebased this, and it should merge now.

pulkitjalan commented 8 years ago

Great, Thanks!

tremby commented 8 years ago

No problem. If you're happy with these two, a release would be great. I also commented on an earlier commit a couple of days ago; did you see that? Not a huge deal.

pulkitjalan commented 8 years ago

Done, yea its now fixed. That was an accident, i was suppose to move it into the data array init. Done that now. Thanks.

tremby commented 8 years ago

Okay, I'm going to put the doc comment about that back in, then. Expect another PR...