Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

3.0.0: Consider removing ip parameter from api #80

Open
zyskowsk opened this issue May 18, 2015 · 1 comment
Open

3.0.0: Consider removing ip parameter from api #80

zyskowsk opened this issue May 18, 2015 · 1 comment
Labels
enhancement
Milestone
3.0

Comments

@zyskowsk
Copy link
Contributor

Remove the ip parameter from event and people methods. Update docs to illustrate how to use optional_params in people methods, as well as how to pass ip in event methods by passing ip in the properties.
@zyskowsk zyskowsk mentioned this issue May 18, 2015
Closed
@zyskowsk zyskowsk added this to the 3.0 milestone Jun 24, 2015
@zyskowsk zyskowsk changed the title 3.0.0: Considering removing ip parameter from api 3.0.0: Consider removing ip parameter from api Jun 24, 2015
@EtienneDepaulis
Copy link
Contributor

👍

Adding some documentation around $ignore_time could also be a great idea :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
3.0
Development

No branches or pull requests
2 participants
@EtienneDepaulis @zyskowsk