search

bradmontgomery / django-redis-metrics

Metrics for django apps backed by Redis.
http://django-redis-metrics.readthedocs.io
MIT License
93 stars 24 forks source link

Allow importing of historical data #44

Closed smaccona closed 9 years ago

smaccona commented 9 years ago

This enables importing of historical data by adding an optional date parameter to the metric and set_metric functions.

Closes #43.

bradmontgomery commented 9 years ago

The tests mock input to redis, so your new date keywarg is what's failing. If you don't mind, just update those calls to the methods you changed in the tests, and this should pass.

smaccona commented 9 years ago

Yes - exactly what I'm doing. I'm also adding an additional test for the new date parameter Thanks!

smaccona commented 9 years ago

@bradmontgomery ready for your review - thanks!

bradmontgomery commented 9 years ago

@smaccona new version is on pypi: https://pypi.python.org/pypi/django-redis-metrics/0.9.0

Thanks!

smaccona commented 9 years ago

Happy to help!