search

pjcalvo / locust-influxdb-listener

LocustIO base project with a custom influxDB listener.
MIT License
26 stars 20 forks source link

Add support for user count over time #17

Closed jmfiola closed 1 year ago

jmfiola commented 1 year ago

This MR adds the "user_count" field to all locust requests

jmfiola commented 1 year ago

@pjcalvo FYI

pjcalvo commented 1 year ago

Hey thanks for doing this PR. A couple of things: The change on the dashboard json is way to big to review. Can u put that on a separate PR, add a more descriptive explanation or attach some screenshots.

Also please remove the increase version. I will increase the version manually when creating a new release tag.

Thanksm

jmfiola commented 1 year ago

Done! No problem, happy to contribute.

jmfiola commented 1 year ago

@pjcalvo Looking for another review here

pjcalvo commented 1 year ago

Thanks, sorry it took me a while to review it.