Skip to content
This repository has been archived by the owner on Mar 25, 2024. It is now read-only.

Dogshell| Enable the type and aggregation_key argument to post event #87

Merged
merged 2 commits into from
May 7, 2014

Conversation

bunelr
Copy link
Contributor

@bunelr bunelr commented May 6, 2014

Fixes #86
Also add the possibility to specify the aggregation_key from the dogshell command line

@@ -146,6 +149,9 @@ def _event(self, title, text, date_happened=None, handle=None, priority=None, re
if aggregation_key is not None:
body['aggregation_key'] = aggregation_key

if event_type is not None:
body['source_type_name'] = event_type
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

source_type_name and event_type are two different fields, why not just name the parameter source_type_name?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You're right, thanks

bunelr added a commit that referenced this pull request May 7, 2014
Dogshell| Enable the type and aggregation_key argument to post event
@bunelr bunelr merged commit 52e1fdb into master May 7, 2014
@wang-arthur wang-arthur deleted the dogshell-args branch December 10, 2014 20:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants