captbaritone / raven-for-redux

A Raven middleware for Redux
295 stars 25 forks source link

Tag Support #69

Closed atticoos closed 6 years ago

atticoos commented 6 years ago

This adds support for tags:

createRavenMiddleware(Raven, {
  getTags: () => ({foo: 'foo', bar: 'bar'})
})

Nothing crazy, it just adds a tags field to data (data.tags = getTags(state))

Screenshot from running this example:

screen shot 2018-04-04 at 4 44 34 pm

codecov-io commented 6 years ago

Codecov Report

Merging #69 into master will not change coverage. The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #69   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           1      1           
  Lines          24     26    +2     
  Branches        9     10    +1     
=====================================
+ Hits           24     26    +2
Impacted Files Coverage Δ
index.js 100% <100%> (ø) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 44292fc...2b7376d. Read the comment docs.

captbaritone commented 6 years ago

Nice looks good! I'll try to find some time to add this to the documentation in the readme, and cut a new release.

captbaritone commented 6 years ago

Actually, would you mind taking a stab at adding some docs to the README.md?

atticoos commented 6 years ago

Happy to -- I originally meant to and totally forgot 😄

Added to my reminders for tomorrow.

captbaritone commented 6 years ago

Shipped in 1.3.1 (I had already published 1.3.0 but forgotten to push the version bump to GitHub)

atticoos commented 6 years ago

Thanks for the ping